Bug 211744

Summary: Remove some unnecessary indirection when getting Document’s Editor
Product: WebKit Reporter: Wenson Hsieh <wenson_hsieh>
Component: HTML EditingAssignee: Wenson Hsieh <wenson_hsieh>
Status: RESOLVED FIXED    
Severity: Normal CC: aboxhall, apinheiro, cdumez, cfleizach, dmazzoni, esprehn+autocc, ews-watchlist, ggaren, jcraig, jdiggs, kangil.han, samuel_white, shihchieh_lee, thorton, webkit-bug-importer, wenson_hsieh
Priority: P2 Keywords: DoNotImportToRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Wenson Hsieh
Reported 2020-05-11 13:55:44 PDT
After r261018, there’s no need to go from Document -> Frame -> Editor if we are already sure that the Frame exists; we can simplify this to just Document -> Editor.
Attachments
Patch (17.43 KB, patch)
2020-05-11 14:11 PDT, Wenson Hsieh 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Wenson Hsieh
Comment 1 2020-05-11 14:11:34 PDT
Created attachment 399055 [details] Patch
Geoffrey Garen
Comment 2 2020-05-11 14:39:16 PDT
Comment on attachment 399055 [details] Patch r=me
Wenson Hsieh
Comment 3 2020-05-11 15:37:23 PDT
Comment on attachment 399055 [details] Patch Thanks for the review!
EWS
Comment 4 2020-05-11 15:44:33 PDT
Committed r261500: <https://trac.webkit.org/changeset/261500> All reviewed patches have been landed. Closing bug and clearing flags on attachment 399055 [details].
Note You need to log in before you can comment on or make changes to this bug.