Temporarily remove the WKHitTestResultRef parameter to avoid breaking the WK2 ABI/API
authorandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 16 Sep 2011 19:29:07 +0000 (19:29 +0000)
committerandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 16 Sep 2011 19:29:07 +0000 (19:29 +0000)
https://bugs.webkit.org/show_bug.cgi?id=68266

Reviewed by John Sullivan.

Source/WebKit2:

* UIProcess/API/C/WKPage.h:
* UIProcess/WebUIClient.cpp:
(WebKit::WebUIClient::mouseDidMoveOverElement):

Tools:

* MiniBrowser/mac/BrowserWindowController.m:
(mouseDidMoveOverElement):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@95311 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/API/C/WKPage.h
Source/WebKit2/UIProcess/WebUIClient.cpp
Tools/ChangeLog
Tools/MiniBrowser/mac/BrowserWindowController.m

index f114b10..3decc23 100644 (file)
@@ -1,3 +1,14 @@
+2011-09-16  Anders Carlsson  <andersca@apple.com>
+
+        Temporarily remove the WKHitTestResultRef parameter to avoid breaking the WK2 ABI/API
+        https://bugs.webkit.org/show_bug.cgi?id=68266
+
+        Reviewed by John Sullivan.
+
+        * UIProcess/API/C/WKPage.h:
+        * UIProcess/WebUIClient.cpp:
+        (WebKit::WebUIClient::mouseDidMoveOverElement):
+
 2011-09-16  Filip Pizlo  <fpizlo@apple.com>
 
         DFG JIT should inline Math.abs
index 67bfb64..bffb96a 100644 (file)
@@ -168,7 +168,7 @@ typedef void (*WKPageTakeFocusCallback)(WKPageRef page, WKFocusDirection directi
 typedef void (*WKPageFocusCallback)(WKPageRef page, const void *clientInfo);
 typedef void (*WKPageUnfocusCallback)(WKPageRef page, const void *clientInfo);
 typedef void (*WKPageSetStatusTextCallback)(WKPageRef page, WKStringRef text, const void *clientInfo);
-typedef void (*WKPageMouseDidMoveOverElementCallback)(WKPageRef page, WKHitTestResultRef hitTestResult, WKEventModifiers modifiers, WKTypeRef userData, const void *clientInfo);
+typedef void (*WKPageMouseDidMoveOverElementCallback)(WKPageRef page, WKEventModifiers modifiers, WKTypeRef userData, const void *clientInfo);
 typedef void (*WKPageMissingPluginButtonClickedCallback)(WKPageRef page, WKStringRef mimeType, WKStringRef url, WKStringRef pluginsPageURL, const void* clientInfo);
 typedef void (*WKPageDidNotHandleKeyEventCallback)(WKPageRef page, WKNativeEventPtr event, const void *clientInfo);
 typedef void (*WKPageDidNotHandleWheelEventCallback)(WKPageRef page, WKNativeEventPtr event, const void *clientInfo);
index 0cbb05d..1dccfcb 100644 (file)
@@ -160,8 +160,9 @@ void WebUIClient::mouseDidMoveOverElement(WebPageProxy* page, const WebHitTestRe
     if (!m_client.mouseDidMoveOverElement)
         return;
 
+    // FIXME: Pass the hit test result to the UI client callback function.
     RefPtr<WebHitTestResult> webHitTestResult = WebHitTestResult::create(data);
-    m_client.mouseDidMoveOverElement(toAPI(page), toAPI(webHitTestResult.get()), toAPI(modifiers), toAPI(userData), m_client.clientInfo);
+    m_client.mouseDidMoveOverElement(toAPI(page), toAPI(modifiers), toAPI(userData), m_client.clientInfo);
 }
 
 void WebUIClient::missingPluginButtonClicked(WebPageProxy* page, const String& mimeType, const String& url, const String& pluginsPageURL)
index 6d31301..16a8d03 100644 (file)
@@ -1,3 +1,13 @@
+2011-09-16  Anders Carlsson  <andersca@apple.com>
+
+        Temporarily remove the WKHitTestResultRef parameter to avoid breaking the WK2 ABI/API
+        https://bugs.webkit.org/show_bug.cgi?id=68266
+
+        Reviewed by John Sullivan.
+
+        * MiniBrowser/mac/BrowserWindowController.m:
+        (mouseDidMoveOverElement):
+
 2011-09-16  Vincent Scheib  <scheib@chromium.org>
 
         Add Vincent Scheib to the committers list.
index 45586cc..f7e6151 100644 (file)
@@ -490,7 +490,7 @@ static void setStatusText(WKPageRef page, WKStringRef text, const void* clientIn
     LOG(@"setStatusText");
 }
 
-static void mouseDidMoveOverElement(WKPageRef page, WKHitTestResultRef hitTestResult, WKEventModifiers modifiers, WKTypeRef userData, const void *clientInfo)
+static void mouseDidMoveOverElement(WKPageRef page, WKEventModifiers modifiers, WKTypeRef userData, const void *clientInfo)
 {
     LOG(@"mouseDidMoveOverElement");
 }