Bug 210560

Summary: Add a test to ensure WebKit::mayContainEditableElementsInRect() returns true if the document element is editable
Product: WebKit Reporter: Daniel Bates <dbates>
Component: Tools / TestsAssignee: Daniel Bates <dbates>
Status: RESOLVED FIXED    
Severity: Normal CC: simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Add a layout test
none
To Land none

Description Daniel Bates 2020-04-15 11:36:01 PDT
Add a test to ensure WebKit::mayContainEditableElementsInRect() returns true if <html contenteditable="true">.
Comment 1 Daniel Bates 2020-04-15 11:39:05 PDT
Created attachment 396554 [details]
Add a layout test
Comment 2 Daniel Bates 2020-04-18 09:22:32 PDT
EWS failures are clearly unrelated.
Comment 3 Simon Fraser (smfr) 2020-04-18 16:11:53 PDT
Comment on attachment 396554 [details]
Add a layout test

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

> LayoutTests/editing/editable-region/hit-test-editable-document-element.html:13
> +if (window.testRunner)
> +    testRunner.dumpAsText();

You don't need this with a js test.
Comment 4 Daniel Bates 2020-04-18 16:13:13 PDT
Comment on attachment 396554 [details]
Add a layout test

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

Thanks Simon for the review!

>> LayoutTests/editing/editable-region/hit-test-editable-document-element.html:13
>> +    testRunner.dumpAsText();
> 
> You don't need this with a js test.

yep, will remove
Comment 5 Daniel Bates 2020-04-18 16:17:12 PDT
Created attachment 396870 [details]
To Land
Comment 6 Daniel Bates 2020-04-18 16:17:35 PDT
Committed r260326: <https://trac.webkit.org/changeset/260326>
Comment 7 Radar WebKit Bug Importer 2020-04-18 16:18:14 PDT
<rdar://problem/61987019>