Attempt to fix the Apple Sierra Release 32-bit Build following <https://trac.webkit...
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Jan 2017 20:32:49 +0000 (20:32 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Jan 2017 20:32:49 +0000 (20:32 +0000)
(https://bugs.webkit.org/show_bug.cgi?id=166367)

Add HAVE(TOUCH_BAR)-guard around code that messages candidateListTouchBarItem() as
candidateListTouchBarItem() is only defined when building with Touch Bar support.

Also added an inline comment to help demarcate the code when building without Touch Bar
support. We should look to clean up WebViewImpl.mm including extracting out the Touch
Bar code to another file or better demarcating it so as to improve the hackability of
this code and prevent breaking the build when building without Touch Bar support.

* UIProcess/Cocoa/WebViewImpl.mm:
(WebKit::WebViewImpl::handleRequestedCandidates):

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

Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/Cocoa/WebViewImpl.mm

index 9d172b9..95c526d 100644 (file)
@@ -1,3 +1,19 @@
+2017-01-03  Daniel Bates  <dabates@apple.com>
+
+        Attempt to fix the Apple Sierra Release 32-bit Build following <https://trac.webkit.org/changeset/210075>
+        (https://bugs.webkit.org/show_bug.cgi?id=166367)
+
+        Add HAVE(TOUCH_BAR)-guard around code that messages candidateListTouchBarItem() as
+        candidateListTouchBarItem() is only defined when building with Touch Bar support.
+
+        Also added an inline comment to help demarcate the code when building without Touch Bar
+        support. We should look to clean up WebViewImpl.mm including extracting out the Touch
+        Bar code to another file or better demarcating it so as to improve the hackability of
+        this code and prevent breaking the build when building without Touch Bar support.
+
+        * UIProcess/Cocoa/WebViewImpl.mm:
+        (WebKit::WebViewImpl::handleRequestedCandidates):
+
 2017-01-03  Michael Catanzaro  <mcatanzaro@igalia.com> and Carlos Garcia Campos  <cgarcia@igalia.com>
 
         [GTK] Expose WebKitSecurityOrigin API
index c9fc184..0ba0624 100644 (file)
@@ -1195,7 +1195,7 @@ void WebViewImpl::setEditableElementIsFocused(bool editableElementIsFocused)
 }
 
 } // namespace WebKit
-#else
+#else // !HAVE(TOUCH_BAR)
 namespace WebKit {
 
 void WebViewImpl::forceRequestCandidatesForTesting()
@@ -2988,7 +2988,9 @@ void WebViewImpl::handleRequestedCandidates(NSInteger sequenceNumber, NSArray<NS
     WebCore::IntRect offsetSelectionRect = postLayoutData.selectionClipRect;
     offsetSelectionRect.move(0, offsetSelectionRect.height());
 
+#if HAVE(TOUCH_BAR)
     [candidateListTouchBarItem() setCandidates:candidates forSelectedRange:selectedRange inString:postLayoutData.paragraphContextForCandidateRequest rect:offsetSelectionRect view:m_view completionHandler:nil];
+#endif
 }
 
 static WebCore::TextCheckingResult textCheckingResultFromNSTextCheckingResult(NSTextCheckingResult *nsResult)