Changeset 109078 in webkit

Timestamp:

Feb 27, 2012 10:34:10 PM (12 years ago)

Author:

koz@chromium.org

Message:

Add missing parameter to didCreateScriptContext() so it overrides base class again.
​https://bugs.webkit.org/show_bug.cgi?id=79742

Reviewed by Kent Tamura.

• tests/WebFrameTest.cpp:

Location:

trunk/Source/WebKit/chromium

Files:

• ChangeLog (modified) (1 diff)
• tests/WebFrameTest.cpp (modified) (1 diff)

trunk/Source/WebKit/chromium/ChangeLog

@@ +1 @@
+2012-02-27  James Kozianski  <koz@chromium.org>
+
+        Add missing parameter to didCreateScriptContext() so it overrides base class again.
+        https://bugs.webkit.org/show_bug.cgi?id=79742
+
+        Reviewed by Kent Tamura.
+
+        * tests/WebFrameTest.cpp:
+
 2012-02-27  Kenichi Ishibashi  <bashi@chromium.org>
 

trunk/Source/WebKit/chromium/tests/WebFrameTest.cpp

@@ -323 +323 @@
 
  private:
-    virtual void didCreateScriptContext(WebFrame* frame, v8::Handle<v8::Context> context, int worldId)
+    virtual void didCreateScriptContext(WebFrame* frame, v8::Handle<v8::Context> context, int extensionGroup, int worldId) OVERRIDE
     {
         createNotifications.push_back(new Notification(frame, context, worldId));
     }
 
-    virtual void willReleaseScriptContext(WebFrame* frame, v8::Handle<v8::Context> context, int worldId)
+    virtual void willReleaseScriptContext(WebFrame* frame, v8::Handle<v8::Context> context, int worldId) OVERRIDE
     {
         releaseNotifications.push_back(new Notification(frame, context, worldId));