[webkit-reviews] review granted: [Bug 195240] [ContentChangeObserver] Move away from WKContentChange values : [Attachment 363409] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Mar 2 13:18:50 PST 2019

Simon Fraser (smfr) <simon.fraser at apple.com> has granted zalan
<zalan at apple.com>'s request for review:
Bug 195240: [ContentChangeObserver] Move away from WKContentChange values

Attachment 363409: Patch


--- Comment #3 from Simon Fraser (smfr) <simon.fraser at apple.com> ---
Comment on attachment 363409
  --> https://bugs.webkit.org/attachment.cgi?id=363409

View in context: https://bugs.webkit.org/attachment.cgi?id=363409&action=review

> Source/WebCore/ChangeLog:9
> +	   This is in preparation for moving towards a state machine like

state machine-like

> Source/WebCore/page/ios/ContentChangeObserver.cpp:133
> +	   LOG(ContentObservation, "stopObservingStyleRecalc: notify the
pending synthetic click handler.");

Mentioning synthetic clicks here indicates a layering problem, as we talked

> Source/WebCore/page/ios/ContentChangeObserver.cpp:137
> +    LOG(ContentObservation, "stopObservingStyleRecalc: can't decided it

"can't decided it" ?

> Source/WebCore/page/ios/ContentChangeObserver.cpp:193
> +bool ContentChangeObserver::hasDeterminedState() const

So "determined" as an adjective means "intent on" as in "he was very
determined". This should use "determinate", meaning "fixed" or "settled".

More information about the webkit-reviews mailing list