Title: [168745] trunk/Source/WebKit2
Revision
168745
Author
[email protected]
Date
2014-05-13 15:26:43 -0700 (Tue, 13 May 2014)

Log Message

AX: Hit testing not accounting for top content inset.
https://bugs.webkit.org/show_bug.cgi?id=132876

Reviewed by Chris Fleizach.

Updated accessibilityHitTest: to account for top content inset when converting from screen coords to WebKit coords.

* WebProcess/WebPage/mac/WKAccessibilityWebPageObjectMac.mm:
(-[WKAccessibilityWebPageObject accessibilityHitTest:]):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (168744 => 168745)


--- trunk/Source/WebKit2/ChangeLog	2014-05-13 22:13:03 UTC (rev 168744)
+++ trunk/Source/WebKit2/ChangeLog	2014-05-13 22:26:43 UTC (rev 168745)
@@ -1,3 +1,15 @@
+2014-05-13  Samuel White  <[email protected]>
+
+        AX: Hit testing not accounting for top content inset.
+        https://bugs.webkit.org/show_bug.cgi?id=132876
+
+        Reviewed by Chris Fleizach.
+
+        Updated accessibilityHitTest: to account for top content inset when converting from screen coords to WebKit coords.
+
+        * WebProcess/WebPage/mac/WKAccessibilityWebPageObjectMac.mm:
+        (-[WKAccessibilityWebPageObject accessibilityHitTest:]):
+
 2014-05-13  Enrica Casucci  <[email protected]>
 
         REGRESSION (WebKit2): Zooming to text field leaves it partially hidden by the form assistant.

Modified: trunk/Source/WebKit2/WebProcess/WebPage/mac/WKAccessibilityWebPageObjectMac.mm (168744 => 168745)


--- trunk/Source/WebKit2/WebProcess/WebPage/mac/WKAccessibilityWebPageObjectMac.mm	2014-05-13 22:13:03 UTC (rev 168744)
+++ trunk/Source/WebKit2/WebProcess/WebPage/mac/WKAccessibilityWebPageObjectMac.mm	2014-05-13 22:26:43 UTC (rev 168745)
@@ -199,7 +199,9 @@
     IntPoint convertedPoint = m_page->screenToRootView(IntPoint(point));
     if (WebCore::FrameView* frameView = m_page->mainFrameView())
         convertedPoint.moveBy(frameView->scrollPosition());
-        
+    if (WebCore::Page* page = m_page->corePage())
+        convertedPoint.move(0, -page->topContentInset());
+    
     return [[self accessibilityRootObjectWrapper] accessibilityHitTest:convertedPoint];
 }
 #pragma clang diagnostic pop
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to