Summary: | [WebKitLegacy] Implement -hidePlaceholder and -showPlaceholderIfNecessary in terms of setCanShowPlaceholder() | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Daniel Bates <dbates> | ||||||
Component: | WebCore Misc. | Assignee: | Daniel Bates <dbates> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | cdumez, eric.carlson, esprehn+autocc, ews-watchlist, gyuyoung.kim, mifenton, simon.fraser, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Local Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Daniel Bates
2020-04-28 12:11:09 PDT
Created attachment 397871 [details]
Patch
Window failure unrelated. Comment on attachment 397871 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=397871&action=review > Source/WebKitLegacy/mac/ChangeLog:11 > + 1. Unlike -hidePlaceholder and -showPlaceholderIfNecessary, setCanShowPlaceholder() does NOT of So much indent. Created attachment 397901 [details]
To Land - now with 50% less indentation
Committed r260854: <https://trac.webkit.org/changeset/260854> Comment on attachment 397901 [details] To Land - now with 50% less indentation View in context: https://bugs.webkit.org/attachment.cgi?id=397901&action=review > Source/WebKitLegacy/mac/ChangeLog:11 > + 1. Unlike -hidePlaceholder and -showPlaceholderIfNecessary, setCanShowPlaceholder() does NOT of s/does NOT of/does Not/ Comment on attachment 397901 [details] To Land - now with 50% less indentation View in context: https://bugs.webkit.org/attachment.cgi?id=397901&action=review >> Source/WebKitLegacy/mac/ChangeLog:11 >> + 1. Unlike -hidePlaceholder and -showPlaceholderIfNecessary, setCanShowPlaceholder() does NOT of > > s/does NOT of/does Not/ yep, I noticed that too AFTER I landed it. I may fix it up in the ChangeLog... |