Attachment 341385 Details for Bug 185897 – patch

[patch] patch

patch.txt (text/plain), 6.68 KB, created by Nan Wang on 2018-05-25 21:21:54 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Nan Wang

Created: 2018-05-25 21:21:54 PDT

Size: 6.68 KB

patch

obsolete

>Index: Source/WebCore/ChangeLog
>===================================================================
>--- Source/WebCore/ChangeLog	(revision 232220)
>+++ Source/WebCore/ChangeLog	(working copy)
>@@ -1,3 +1,18 @@
>+2018-05-25  Nan Wang  <n_wang@apple.com>
>+
>+        AX: setValue on contenteditable should preserve whitespace
>+        https://bugs.webkit.org/show_bug.cgi?id=185897
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        We should mimic typing when setting value to a contenteditable from accessibility
>+        instead of mutating the DOM by using setInnerText.
>+
>+        Updated tests to cover this change.
>+
>+        * accessibility/AccessibilityRenderObject.cpp:
>+        (WebCore::AccessibilityRenderObject::setValue):
>+
> 2018-05-25  Eric Carlson  <eric.carlson@apple.com>
> 
>         Captions are sized incorrectly in PiP mode
>Index: Source/WebCore/accessibility/AccessibilityRenderObject.cpp
>===================================================================
>--- Source/WebCore/accessibility/AccessibilityRenderObject.cpp	(revision 232220)
>+++ Source/WebCore/accessibility/AccessibilityRenderObject.cpp	(working copy)
>@@ -1778,15 +1778,15 @@ void AccessibilityRenderObject::setValue
>     else if (renderer.isTextArea() && is<HTMLTextAreaElement>(element))
>         downcast<HTMLTextAreaElement>(element).setValue(string);
>     else if (is<HTMLElement>(element) && contentEditableAttributeIsEnabled(&element)) {
>-        // Set the style to the element so the child Text node won't collapse spaces
>-        if (is<RenderElement>(renderer)) {
>-            RenderElement& renderElement = downcast<RenderElement>(renderer);
>-            auto style = RenderStyle::create();
>-            style.inheritFrom(renderElement.style());
>-            style.setWhiteSpace(WhiteSpace::Pre);
>-            renderElement.setStyleInternal(WTFMove(style));
>+        // We should use the editor's insertText to mimic typing into the contenteditable field.
>+        // Also only do this when the field is in editing mode.
>+        if (Frame* frame = renderer.document().frame()) {
>+            Editor& editor = frame->editor();
>+            if (element.shouldUseInputMethod()) {
>+                editor.clearText();
>+                editor.insertText(string, nullptr);
>+            }
>         }
>-        downcast<HTMLElement>(element).setInnerText(string);
>     }
> }
> 
>Index: LayoutTests/ChangeLog
>===================================================================
>--- LayoutTests/ChangeLog	(revision 232220)
>+++ LayoutTests/ChangeLog	(working copy)
>@@ -1,3 +1,15 @@
>+2018-05-25  Nan Wang  <n_wang@apple.com>
>+
>+        AX: setValue on contenteditable should preserve whitespace
>+        https://bugs.webkit.org/show_bug.cgi?id=185897
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * accessibility/mac/AOM-event-accessiblesetvalue-expected.txt:
>+        * accessibility/mac/AOM-event-accessiblesetvalue.html:
>+        * accessibility/mac/set-value-editable-types-expected.txt:
>+        * accessibility/mac/set-value-editable-types.html:
>+
> 2018-05-25  Youenn Fablet  <youenn@apple.com>
> 
>         Migrate From-Origin to Cross-Origin-Resource-Policy
>Index: LayoutTests/accessibility/mac/AOM-event-accessiblesetvalue-expected.txt
>===================================================================
>--- LayoutTests/accessibility/mac/AOM-event-accessiblesetvalue-expected.txt	(revision 232220)
>+++ LayoutTests/accessibility/mac/AOM-event-accessiblesetvalue-expected.txt	(working copy)
>@@ -13,6 +13,7 @@ combobox accessible set value to: combob
> 
> Test Contenteditable.
> contenteditable accessible set value to: contenteditable new value
>+PASS axNode.stringValue is 'AXValue: contenteditable new value'
> 
> Test Slider.
> slider accessible set value to: 70
>Index: LayoutTests/accessibility/mac/AOM-event-accessiblesetvalue.html
>===================================================================
>--- LayoutTests/accessibility/mac/AOM-event-accessiblesetvalue.html	(revision 232220)
>+++ LayoutTests/accessibility/mac/AOM-event-accessiblesetvalue.html	(working copy)
>@@ -63,8 +63,16 @@
>            axNode = accessibilityController.accessibleElementById("contenteditable");
>            node.onaccessiblesetvalue = function(event) {
>               debug("contenteditable accessible set value to: " + event.value);
>-              testSlider();
>            };
>+           accessibilityController.addNotificationListener(function(element, notification) {
>+               if (notification == "AXValueChanged") {
>+                   shouldBe("axNode.stringValue", "'AXValue: contenteditable new value'");
>+                   node.blur();
>+                   accessibilityController.removeNotificationListener();
>+                   testSlider();
>+               }
>+           });
>+           node.focus();
>            axNode.setValue("contenteditable new value");
>        }
>        
>Index: LayoutTests/accessibility/mac/set-value-editable-types-expected.txt
>===================================================================
>--- LayoutTests/accessibility/mac/set-value-editable-types-expected.txt	(revision 232220)
>+++ LayoutTests/accessibility/mac/set-value-editable-types-expected.txt	(working copy)
>@@ -9,7 +9,7 @@ Role: AXRole: AXTextArea
> Value: AXValue: current1
> Writable: true
> Value change notification received
>-Updated Value: AXValue:    leading and trailing spaces   
>+Updated Value: AXValue: Â  Â leading and trailing spaces Â Â 
> PASS successfullyParsed is true
> 
> TEST COMPLETE
>Index: LayoutTests/accessibility/mac/set-value-editable-types.html
>===================================================================
>--- LayoutTests/accessibility/mac/set-value-editable-types.html	(revision 232220)
>+++ LayoutTests/accessibility/mac/set-value-editable-types.html	(working copy)
>@@ -37,6 +37,8 @@
>             var writable = axElement.isAttributeSettable("AXValue");
>             debug("Writable: " + writable);
> 
>+            document.getElementById(idValue).focus();
>+
>             axElement.setValue("   leading and trailing spaces   ");
>         }
> 
>Index: LayoutTests/platform/mac-wk1/TestExpectations
>===================================================================
>--- LayoutTests/platform/mac-wk1/TestExpectations	(revision 232220)
>+++ LayoutTests/platform/mac-wk1/TestExpectations	(working copy)
>@@ -514,6 +514,8 @@ webkit.org/b/180898 accessibility/mac/AO
> webkit.org/b/183023 accessibility/mac/AOM-events-all.html [ Skip ]
> webkit.org/b/183352 accessibility/ios-simulator/AOM-dismiss-event.html [ Skip ]
> webkit.org/b/184742 accessibility/mac/async-increment-decrement-action.html [ Skip ]
>+webkit.org/b/185897 accessibility/mac/AOM-event-accessiblesetvalue.html [ Skip ]
>+webkit.org/b/185897 accessibility/mac/set-value-editable-types.html [ Skip ]
> 
> webkit.org/b/182752 accessibility/mac/accessibility-make-first-responder.html [ Skip ]
>

Flags:

rniwa: review+

Actions: View | Formatted Diff | Diff

Attachments on bug 185897: 341053 | 341054 | 341062 | 341068 | 341369 | 341376 | 341377 | 341385