[chromium] Plumb extensionGroup into didCreateScriptContext().
Created attachment 127906 [details] Patch
Please wait for approval from fishd@chromium.org before submitting because this patch contains changes to the Chromium public API.
Comment on attachment 127906 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=127906&action=review > Source/WebKit/chromium/public/WebFrameClient.h:300 > + virtual void didCreateScriptContext(WebFrame*, v8::Handle<v8::Context>, int extensionGroup, int worldId) { } Is your plan to land a Chromium change ahead of this change that implements this new method signature?
(In reply to comment #3) > (From update of attachment 127906 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=127906&action=review > > > Source/WebKit/chromium/public/WebFrameClient.h:300 > > + virtual void didCreateScriptContext(WebFrame*, v8::Handle<v8::Context>, int extensionGroup, int worldId) { } > > Is your plan to land a Chromium change ahead of this change that implements this new method signature? Yep, that's right.
Created attachment 129120 [details] Patch for landing
Comment on attachment 129120 [details] Patch for landing Clearing flags on attachment: 129120 Committed r109064: <http://trac.webkit.org/changeset/109064>
All reviewed patches have been landed. Closing bug.