Bug 166367 – Holding down on candidates in the TouchBar should show panel on screen

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit Misc.

[Beth Dakin]

Holding down on candidates in the TouchBar should show panel on screen

rdar://problem/28479236

[Beth Dakin]

Created attachment 297619 [details]
Patch

[Beth Dakin]

Thanks Tim! https://trac.webkit.org/changeset/210075

[Daniel Bates]

(In reply to comment #2)
> Thanks Tim! https://trac.webkit.org/changeset/210075

This broke the Apple Sierra Release 32-bit build, which builds without Touch Bar support:

[[
release/build/Source/WebKit2/UIProcess/Cocoa/WebViewImpl.mm:2991:6: error: use of undeclared identifier 'candidateListTouchBarItem'
    [candidateListTouchBarItem() setCandidates:candidates forSelectedRange:selectedRange inString:postLayoutData.paragraphContextForCandidateRequest rect:offsetSelectionRect view:m_view completionHandler:nil];
     ^
]]
<https://build.webkit.org/builders/Apple%20Sierra%20Release%20%2832-bit%20Build%29/builds/2528/steps/compile-webkit/logs/stdio>

[Daniel Bates]

(In reply to comment #3)
> (In reply to comment #2)
> > Thanks Tim! https://trac.webkit.org/changeset/210075
> 
> This broke the Apple Sierra Release 32-bit build, which builds without Touch
> Bar support:
> 
> [[
> release/build/Source/WebKit2/UIProcess/Cocoa/WebViewImpl.mm:2991:6: error:
> use of undeclared identifier 'candidateListTouchBarItem'
>     [candidateListTouchBarItem() setCandidates:candidates
> forSelectedRange:selectedRange
> inString:postLayoutData.paragraphContextForCandidateRequest
> rect:offsetSelectionRect view:m_view completionHandler:nil];
>      ^
> ]]
> <https://build.webkit.org/builders/Apple%20Sierra%20Release%20%2832-
> bit%20Build%29/builds/2528/steps/compile-webkit/logs/stdio>

Committed build fix in <https://trac.webkit.org/changeset/210245>.

[Daniel Bates]

We should look to extract out the Touch Bar code from Source/WebKit2/UIProcess/Cocoa/WebViewImpl.mm or at least consolidate it at one end of this file. Currently the Touch Bar code is scattered throughout this file and in #if/else chunks that are many lines long. As demonstrated by the Apple Sierra Release 32-bit build failure (comment #3) it is easy to lose track of what chunk in the file is HAVE(TOUCH_BAR)-guarded or not. I needed to use an editor that supports collapsing code regions in order to be able to track the #if HAVE(TOUCH_BAR) and #else chunks. This significantly affects the hackability of this file and makes refactoring code in it error prone.

[Daniel Bates]

Committed another attempt to fix the Apple Sierra Release 32-bit build in <https://trac.webkit.org/changeset/210247>.

[Alexey Proskuryakov]

We need to update Xcode on this bot; I don't think that this is a supported configuration any more.

[Alexey Proskuryakov]

Ryan took care of updating it.