Bug 189295

Summary: [macOS] Cannot change font size at selection until font panel is shown
Product: WebKit Reporter: Wenson Hsieh <wenson_hsieh>
Component: HTML EditingAssignee: Wenson Hsieh <wenson_hsieh>
Status: RESOLVED FIXED    
Severity: Normal CC: bdakin, commit-queue, ews-watchlist, mitz, rniwa, thorton, webkit-bug-importer, wenson_hsieh
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 189356    
Attachments:
Description Flags
WIP
none
WIP 2
none
First pass
none
Rebase on trunk
none
Try to fix the build
none
Fix 32-bit macOS build
none
Archive of layout-test-results from ews107 for mac-sierra-wk2
none
Patch for EWS
none
Patch for landing none

Description Wenson Hsieh 2018-09-04 21:24:19 PDT
<rdar://problem/35593389>
Comment 1 Wenson Hsieh 2018-09-04 21:33:57 PDT
Created attachment 348892 [details]
WIP
Comment 2 Wenson Hsieh 2018-09-05 11:19:48 PDT
Created attachment 348947 [details]
WIP 2
Comment 3 Wenson Hsieh 2018-09-05 17:00:35 PDT
Created attachment 348981 [details]
First pass
Comment 4 Wenson Hsieh 2018-09-05 17:24:42 PDT
Created attachment 348984 [details]
Rebase on trunk
Comment 5 Wenson Hsieh 2018-09-05 17:46:24 PDT
Created attachment 348986 [details]
Try to fix the build
Comment 6 Wenson Hsieh 2018-09-05 18:07:05 PDT
Created attachment 348989 [details]
Fix 32-bit macOS build
Comment 7 EWS Watchlist 2018-09-05 19:42:17 PDT
Comment on attachment 348989 [details]
Fix 32-bit macOS build

Attachment 348989 [details] did not pass mac-wk2-ews (mac-wk2):
Output: https://webkit-queues.webkit.org/results/9108959

New failing tests:
css3/filters/backdrop/add-remove-add-backdrop-filter.html
Comment 8 EWS Watchlist 2018-09-05 19:42:19 PDT
Created attachment 348995 [details]
Archive of layout-test-results from ews107 for mac-sierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107  Port: mac-sierra-wk2  Platform: Mac OS X 10.12.6
Comment 9 Ryosuke Niwa 2018-09-05 22:04:58 PDT
Comment on attachment 348989 [details]
Fix 32-bit macOS build

r=me. This patch probably didn't introduce a regression in backdrop filter LOL.
Comment 10 Wenson Hsieh 2018-09-06 07:29:45 PDT
(In reply to Ryosuke Niwa from comment #9)
> Comment on attachment 348989 [details]
> Fix 32-bit macOS build
> 
> r=me. This patch probably didn't introduce a regression in backdrop filter
> LOL.

Yeah, this backdrop filter test passes locally for me (and I've also seen it flaky on other unrelated patches on EWS), but I'll do another EWS run to be on the safe side.
Comment 11 Wenson Hsieh 2018-09-06 08:45:36 PDT
Created attachment 349029 [details]
Patch for EWS
Comment 12 Wenson Hsieh 2018-09-06 12:08:34 PDT
Created attachment 349050 [details]
Patch for landing
Comment 13 WebKit Commit Bot 2018-09-06 12:46:45 PDT
Comment on attachment 349050 [details]
Patch for landing

Clearing flags on attachment: 349050

Committed r235748: <https://trac.webkit.org/changeset/235748>
Comment 14 WebKit Commit Bot 2018-09-06 12:46:47 PDT
All reviewed patches have been landed.  Closing bug.