Opened 5 years ago

Closed 5 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):

http://codereview.adblockplus.org/6112412478472192/

Description

Background

Recent v8 api required to pass isolate in most functions. For example, it does not support constructing of v8::String without specifying isolate anymore https://code.google.com/p/v8/source/browse/trunk/include/v8.h?r=25122#2012 .

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 5 years ago by sergz

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

comment:2 Changed 5 years ago by sergz

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