Summary: | Remove redundant renderObject in BreakingContext::handleText | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | zalan <zalan> | ||||||
Component: | Layout and Rendering | Assignee: | zalan <zalan> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bfulgham, changseok, darin, esprehn+autocc, ews-watchlist, glenn, kondapallykalyan, pdr, simon.fraser, webkit-bug-importer, zalan | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
zalan
2021-04-16 11:00:42 PDT
Created attachment 426247 [details]
Patch
Comment on attachment 426247 [details]
Patch
I would name this local variable "renderer". Sure, its type is RenderText&, but the role of the object is "the renderer for the text we are handling".
(In reply to Darin Adler from comment #2) > Comment on attachment 426247 [details] > Patch > > I would name this local variable "renderer". Sure, its type is RenderText&, > but the role of the object is "the renderer for the text we are handling". ok Created attachment 426249 [details]
Patch
Committed r276169 (236656@main): <https://commits.webkit.org/236656@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 426249 [details]. |