Title: [113688] trunk/Source/WebKit/chromium
Revision
113688
Author
[email protected]
Date
2012-04-09 23:07:17 -0700 (Mon, 09 Apr 2012)

Log Message

[Chromium] Remove unused variable, WebViewImpl::m_lastMousePosition.
https://bugs.webkit.org/show_bug.cgi?id=83550

Reviewed by Kentaro Hara.

* src/WebViewImpl.cpp:
(WebKit::WebViewImpl::WebViewImpl):
(WebKit::WebViewImpl::mouseMove):
* src/WebViewImpl.h:

Modified Paths

Diff

Modified: trunk/Source/WebKit/chromium/ChangeLog (113687 => 113688)


--- trunk/Source/WebKit/chromium/ChangeLog	2012-04-10 05:54:11 UTC (rev 113687)
+++ trunk/Source/WebKit/chromium/ChangeLog	2012-04-10 06:07:17 UTC (rev 113688)
@@ -1,3 +1,15 @@
+2012-04-09  Kent Tamura  <[email protected]>
+
+        [Chromium] Remove unused variable, WebViewImpl::m_lastMousePosition.
+        https://bugs.webkit.org/show_bug.cgi?id=83550
+
+        Reviewed by Kentaro Hara.
+
+        * src/WebViewImpl.cpp:
+        (WebKit::WebViewImpl::WebViewImpl):
+        (WebKit::WebViewImpl::mouseMove):
+        * src/WebViewImpl.h:
+
 2012-04-09  Dana Jansens  <[email protected]>
 
         [chromium] Viewport is not filled when out of texture memory on mac

Modified: trunk/Source/WebKit/chromium/src/WebViewImpl.cpp (113687 => 113688)


--- trunk/Source/WebKit/chromium/src/WebViewImpl.cpp	2012-04-10 05:54:11 UTC (rev 113687)
+++ trunk/Source/WebKit/chromium/src/WebViewImpl.cpp	2012-04-10 06:07:17 UTC (rev 113688)
@@ -398,9 +398,6 @@
     WTF::initializeThreading();
     WTF::initializeMainThread();
 
-    // set to impossible point so we always get the first mouse pos
-    m_lastMousePosition = WebPoint(-1, -1);
-
     Page::PageClients pageClients;
     pageClients.chromeClient = &m_chromeClientImpl;
     pageClients.contextMenuClient = &m_contextMenuClientImpl;
@@ -469,8 +466,6 @@
     if (!mainFrameImpl() || !mainFrameImpl()->frameView())
         return;
 
-    m_lastMousePosition = WebPoint(event.x, event.y);
-
     // We call mouseMoved here instead of handleMouseMovedEvent because we need
     // our ChromeClientImpl to receive changes to the mouse position and
     // tooltip text, and mouseMoved handles all of that.

Modified: trunk/Source/WebKit/chromium/src/WebViewImpl.h (113687 => 113688)


--- trunk/Source/WebKit/chromium/src/WebViewImpl.h	2012-04-10 05:54:11 UTC (rev 113687)
+++ trunk/Source/WebKit/chromium/src/WebViewImpl.h	2012-04-10 06:07:17 UTC (rev 113688)
@@ -611,7 +611,6 @@
     // The upper bound on the size when auto-resizing.
     WebCore::IntSize m_maxAutoSize;
 
-    WebPoint m_lastMousePosition;
     OwnPtr<WebCore::Page> m_page;
 
     // This flag is set when a new navigation is detected. It is used to satisfy
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to