Title: [178676] trunk/Source/WebKit/mac
Revision
178676
Author
bda...@apple.com
Date
2015-01-19 15:24:38 -0800 (Mon, 19 Jan 2015)

Log Message

REGRESSION (r178290): Yellow highlight from context menu Lookup in Dictionary is 
offset by flipped-ness
https://bugs.webkit.org/show_bug.cgi?id=140643
-and corresponding-
rdar://problem/19489593

Reviewed by Tim Horton.

We universally flipped because we assumed that the root view was flipped and the 
WebView was not. However, in Dictionary, the WebView is flipped! So this patch 
fixes that assumption by checking the WebView’s flipped-ness.
* WebView/WebView.mm:
(-[WebView _convertRectFromRootView:]):

Modified Paths

Diff

Modified: trunk/Source/WebKit/mac/ChangeLog (178675 => 178676)


--- trunk/Source/WebKit/mac/ChangeLog	2015-01-19 23:09:36 UTC (rev 178675)
+++ trunk/Source/WebKit/mac/ChangeLog	2015-01-19 23:24:38 UTC (rev 178676)
@@ -1,3 +1,19 @@
+2015-01-19  Beth Dakin  <bda...@apple.com>
+
+        REGRESSION (r178290): Yellow highlight from context menu Lookup in Dictionary is 
+        offset by flipped-ness
+        https://bugs.webkit.org/show_bug.cgi?id=140643
+        -and corresponding-
+        rdar://problem/19489593
+
+        Reviewed by Tim Horton.
+
+        We universally flipped because we assumed that the root view was flipped and the 
+        WebView was not. However, in Dictionary, the WebView is flipped! So this patch 
+        fixes that assumption by checking the WebView’s flipped-ness.
+        * WebView/WebView.mm:
+        (-[WebView _convertRectFromRootView:]):
+
 2015-01-19  Commit Queue  <commit-qu...@webkit.org>
 
         Unreviewed, rolling out r178653.

Modified: trunk/Source/WebKit/mac/WebView/WebView.mm (178675 => 178676)


--- trunk/Source/WebKit/mac/WebView/WebView.mm	2015-01-19 23:09:36 UTC (rev 178675)
+++ trunk/Source/WebKit/mac/WebView/WebView.mm	2015-01-19 23:24:38 UTC (rev 178676)
@@ -8572,6 +8572,8 @@
 
 - (NSRect)_convertRectFromRootView:(NSRect)rect
 {
+    if (self.isFlipped)
+        return rect;
     return NSMakeRect(rect.origin.x, [self bounds].size.height - rect.origin.y - rect.size.height, rect.size.width, rect.size.height);
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to