[webkit-reviews] review denied: [Bug 190977] '-webkit-user-select: all' is not respected in editable views : [Attachment 353209] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Oct 28 11:07:15 PDT 2018


Ryosuke Niwa <rniwa at webkit.org> has denied Tim Horton <thorton at apple.com>'s
request for review:
Bug 190977: '-webkit-user-select: all' is not respected in editable views
https://bugs.webkit.org/show_bug.cgi?id=190977

Attachment 353209: Patch

https://bugs.webkit.org/attachment.cgi?id=353209&action=review




--- Comment #19 from Ryosuke Niwa <rniwa at webkit.org> ---
Comment on attachment 353209
  --> https://bugs.webkit.org/attachment.cgi?id=353209
Patch

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

> Source/WebCore/dom/Node.cpp:751
> +	   if (documentIsEditable)
> +	       return Node::Editability::CanEditRichly;
> +

This isn't right. It's totally possible to have -webkit-user-modify: read-only
inside a design mode document.
As done in editabilityFromContentEditableAttr, we need to wait until we get out
of this loop.
r- because of this and the test failures.


More information about the webkit-reviews mailing list