Opened 6 years ago

Closed 6 years ago

#1547 closed change (fixed)

Pass isolate to v8::API

Reported by: sergz Assignee:
Priority: Unknown Milestone:
Module: Libadblockplus Keywords:
Cc: Blocked By:
Blocking: #1197 Platform: Unknown
Ready: no Confidential: no
Tester: Verified working: no
Review URL(s):



Recent v8 api required to pass isolate in most functions. For example, it does not support constructing of v8::String without specifying isolate anymore .

The tendency seems to get rid of v8::Isolate::GetCurrent(), so it looks better for future to avoid such calls.

What to change

Add v8::Isolate* argument and adapt all calls of AdblockPlus::Utils::ToV8String and to other calls.

Change History (2)

comment:1 Changed 6 years ago by sergz

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:2 Changed 6 years ago by sergz

  • Resolution set to fixed
  • Status changed from reviewing to closed
Note: See TracTickets for help on using tickets.