Reduce code duplication in WebView::updateNativeCursor
authoraroben@apple.com <aroben@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Jan 2011 20:49:02 +0000 (20:49 +0000)
committeraroben@apple.com <aroben@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Jan 2011 20:49:02 +0000 (20:49 +0000)
Fixes <http://webkit.org/b/52005> It's not easy to add new behavior to
WebView::updateNativeCursor

Reviewed by Ada Chan.

* UIProcess/win/WebView.cpp:
(WebKit::WebView::cursorToShow): Moved code to choose a cursor here...
(WebKit::WebView::updateNativeCursor): ...from here.

* UIProcess/win/WebView.h: Added cursorToShow.

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

WebKit2/ChangeLog
WebKit2/UIProcess/win/WebView.cpp
WebKit2/UIProcess/win/WebView.h

index dc523ac4f6b84dd647d8f363bcabc1f9ae70c8ed..3534d75a8a2a3b0c8cbd83abd21863a39879c148 100644 (file)
@@ -1,3 +1,18 @@
+2011-01-06  Adam Roben  <aroben@apple.com>
+
+        Reduce code duplication in WebView::updateNativeCursor
+
+        Fixes <http://webkit.org/b/52005> It's not easy to add new behavior to
+        WebView::updateNativeCursor
+
+        Reviewed by Ada Chan.
+
+        * UIProcess/win/WebView.cpp:
+        (WebKit::WebView::cursorToShow): Moved code to choose a cursor here...
+        (WebKit::WebView::updateNativeCursor): ...from here.
+
+        * UIProcess/win/WebView.h: Added cursorToShow.
+
 2011-01-06  Adam Roben  <aroben@apple.com>
 
         Make WKView fill with white when the web process has crashed
index 4c2dc9f4a66bcb9af1a9cb790bdd59e3139ea027..d4d77bed566998508632ebf54dbf45e38fefe4e3 100644 (file)
@@ -622,15 +622,19 @@ void WebView::toolTipChanged(const String&, const String& newToolTip)
     ::SendMessage(m_toolTipWindow, TTM_ACTIVATE, !newToolTip.isEmpty(), 0);
 }
 
-void WebView::updateNativeCursor()
+HCURSOR WebView::cursorToShow() const
 {
     // We only show the override cursor if the default (arrow) cursor is showing.
     static HCURSOR arrowCursor = ::LoadCursor(0, IDC_ARROW);
     if (m_overrideCursor && m_webCoreCursor == arrowCursor)
-        m_lastCursorSet = m_overrideCursor;
-    else
-        m_lastCursorSet = m_webCoreCursor;
+        return m_overrideCursor;
+
+    return m_webCoreCursor;
+}
 
+void WebView::updateNativeCursor()
+{
+    m_lastCursorSet = cursorToShow();
     ::SetCursor(m_lastCursorSet);
 }
 
index 814fed4bb07de7f38200b948807220637ed6d5f9..321615408933e4c039301f14efdbd3a48aaae941 100644 (file)
@@ -101,6 +101,7 @@ private:
 
     void close();
 
+    HCURSOR cursorToShow() const;
     void updateNativeCursor();
 
     // PageClient