Bug 211744

Summary:

Remove some unnecessary indirection when getting Document’s Editor

Product:

WebKit

Reporter:

Wenson Hsieh <wenson_hsieh>

Component:

HTML Editing

Assignee:

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:

Keywords:

DoNotImportToRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none

Description Wenson Hsieh 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.

Comment 1 Wenson Hsieh 2020-05-11 14:11:34 PDT

Created attachment 399055 [details]
Patch

Comment 2 Geoffrey Garen 2020-05-11 14:39:16 PDT

Comment on attachment 399055 [details]
Patch

r=me

Comment 3 Wenson Hsieh 2020-05-11 15:37:23 PDT

Comment on attachment 399055 [details]
Patch

Thanks for the review!

Comment 4 EWS 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].