Title: [176811] trunk/Source
Revision
176811
Author
[email protected]
Date
2014-12-04 13:10:49 -0800 (Thu, 04 Dec 2014)

Log Message

Fix the 32-bit build.

* UIProcess/API/mac/WKView.mm:
(-[WKView _setTextIndicator:fadeOut:animationCompletionHandler:]):

* WebView/WebView.mm:
(-[WebView _setTextIndicator:fadeOut:animationCompletionHandler:]):

* page/mac/TextIndicatorWindow.h:
* page/mac/TextIndicatorWindow.mm:
(WebCore::TextIndicatorWindow::setTextIndicator):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (176810 => 176811)


--- trunk/Source/WebCore/ChangeLog	2014-12-04 19:58:14 UTC (rev 176810)
+++ trunk/Source/WebCore/ChangeLog	2014-12-04 21:10:49 UTC (rev 176811)
@@ -1,3 +1,11 @@
+2014-12-04  Timothy Horton  <[email protected]>
+
+        Fix the 32-bit build.
+
+        * page/mac/TextIndicatorWindow.h:
+        * page/mac/TextIndicatorWindow.mm:
+        (WebCore::TextIndicatorWindow::setTextIndicator):
+
 2014-12-04  Tim Horton  <[email protected]>
 
         TextIndicator::createWithSelectionInFrame does synchronous IPC in WebKit2

Modified: trunk/Source/WebCore/page/mac/TextIndicatorWindow.h (176810 => 176811)


--- trunk/Source/WebCore/page/mac/TextIndicatorWindow.h	2014-12-04 19:58:14 UTC (rev 176810)
+++ trunk/Source/WebCore/page/mac/TextIndicatorWindow.h	2014-12-04 21:10:49 UTC (rev 176811)
@@ -48,7 +48,7 @@
     explicit TextIndicatorWindow(NSView *);
     ~TextIndicatorWindow();
 
-    void setTextIndicator(PassRefPtr<TextIndicator>, NSRect contentRect, bool fadeOut, std::function<void ()> animationCompletionHandler);
+    void setTextIndicator(PassRefPtr<TextIndicator>, CGRect contentRect, bool fadeOut, std::function<void ()> animationCompletionHandler);
 
 private:
     void closeWindow();

Modified: trunk/Source/WebCore/page/mac/TextIndicatorWindow.mm (176810 => 176811)


--- trunk/Source/WebCore/page/mac/TextIndicatorWindow.mm	2014-12-04 19:58:14 UTC (rev 176810)
+++ trunk/Source/WebCore/page/mac/TextIndicatorWindow.mm	2014-12-04 21:10:49 UTC (rev 176811)
@@ -266,7 +266,7 @@
     closeWindow();
 }
 
-void TextIndicatorWindow::setTextIndicator(PassRefPtr<TextIndicator> textIndicator, NSRect contentRect, bool fadeOut, std::function<void ()> animationCompletionHandler)
+void TextIndicatorWindow::setTextIndicator(PassRefPtr<TextIndicator> textIndicator, CGRect contentRect, bool fadeOut, std::function<void ()> animationCompletionHandler)
 {
     if (m_textIndicator == textIndicator)
         return;
@@ -282,7 +282,7 @@
     CGFloat horizontalMargin = std::max(dropShadowBlurRadius * 2 + horizontalBorder, contentRect.size.width * 2);
     CGFloat verticalMargin = std::max(dropShadowBlurRadius * 2 + verticalBorder, contentRect.size.height * 2);
 
-    contentRect = NSInsetRect(contentRect, -horizontalMargin, -verticalMargin);
+    contentRect = NSInsetRect(NSRectFromCGRect(contentRect), -horizontalMargin, -verticalMargin);
     NSRect windowContentRect = [NSWindow contentRectForFrameRect:NSIntegralRect(contentRect) styleMask:NSBorderlessWindowMask];
     m_textIndicatorWindow = adoptNS([[NSWindow alloc] initWithContentRect:windowContentRect styleMask:NSBorderlessWindowMask backing:NSBackingStoreBuffered defer:NO]);
 

Modified: trunk/Source/WebKit/mac/ChangeLog (176810 => 176811)


--- trunk/Source/WebKit/mac/ChangeLog	2014-12-04 19:58:14 UTC (rev 176810)
+++ trunk/Source/WebKit/mac/ChangeLog	2014-12-04 21:10:49 UTC (rev 176811)
@@ -1,3 +1,10 @@
+2014-12-04  Timothy Horton  <[email protected]>
+
+        Fix the 32-bit build.
+
+        * WebView/WebView.mm:
+        (-[WebView _setTextIndicator:fadeOut:animationCompletionHandler:]):
+
 2014-12-04  Tim Horton  <[email protected]>
 
         TextIndicator::createWithSelectionInFrame does synchronous IPC in WebKit2

Modified: trunk/Source/WebKit/mac/WebView/WebView.mm (176810 => 176811)


--- trunk/Source/WebKit/mac/WebView/WebView.mm	2014-12-04 19:58:14 UTC (rev 176810)
+++ trunk/Source/WebKit/mac/WebView/WebView.mm	2014-12-04 21:10:49 UTC (rev 176811)
@@ -8602,7 +8602,7 @@
         _private->textIndicatorWindow = std::make_unique<TextIndicatorWindow>(self);
 
     NSRect contentRect = [self.window convertRectToScreen:textIndicator->textBoundingRectInWindowCoordinates()];
-    _private->textIndicatorWindow->setTextIndicator(textIndicator, contentRect, fadeOut, WTF::move(completionHandler));
+    _private->textIndicatorWindow->setTextIndicator(textIndicator, NSRectToCGRect(contentRect), fadeOut, WTF::move(completionHandler));
 }
 
 - (void)_clearTextIndicator

Modified: trunk/Source/WebKit2/ChangeLog (176810 => 176811)


--- trunk/Source/WebKit2/ChangeLog	2014-12-04 19:58:14 UTC (rev 176810)
+++ trunk/Source/WebKit2/ChangeLog	2014-12-04 21:10:49 UTC (rev 176811)
@@ -1,3 +1,10 @@
+2014-12-04  Timothy Horton  <[email protected]>
+
+        Fix the 32-bit build.
+
+        * UIProcess/API/mac/WKView.mm:
+        (-[WKView _setTextIndicator:fadeOut:animationCompletionHandler:]):
+
 2014-12-04  Tim Horton  <[email protected]>
 
         TextIndicator::createWithSelectionInFrame does synchronous IPC in WebKit2

Modified: trunk/Source/WebKit2/UIProcess/API/mac/WKView.mm (176810 => 176811)


--- trunk/Source/WebKit2/UIProcess/API/mac/WKView.mm	2014-12-04 19:58:14 UTC (rev 176810)
+++ trunk/Source/WebKit2/UIProcess/API/mac/WKView.mm	2014-12-04 21:10:49 UTC (rev 176811)
@@ -3082,7 +3082,7 @@
         _data->_textIndicatorWindow = std::make_unique<TextIndicatorWindow>(self);
 
     NSRect contentRect = [self.window convertRectToScreen:[self convertRect:textIndicator->textBoundingRectInWindowCoordinates() toView:nil]];
-    _data->_textIndicatorWindow->setTextIndicator(textIndicator, contentRect, fadeOut, WTF::move(completionHandler));
+    _data->_textIndicatorWindow->setTextIndicator(textIndicator, NSRectToCGRect(contentRect), fadeOut, WTF::move(completionHandler));
 }
 
 - (void)_setTextIndicator:(PassRefPtr<TextIndicator>)textIndicator fadeOut:(BOOL)fadeOut
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to