Title: [220535] trunk
Revision
220535
Author
commit-qu...@webkit.org
Date
2017-08-10 11:27:01 -0700 (Thu, 10 Aug 2017)

Log Message

Unreviewed, rolling out r220463.
https://bugs.webkit.org/show_bug.cgi?id=175435

unaccepted fix (Requested by n_wang on #webkit).

Reverted changeset:

"AX: crash at
WebCore::AccessibilityObject::supportsARIALiveRegion() const +
24"
https://bugs.webkit.org/show_bug.cgi?id=175340
http://trac.webkit.org/changeset/220463

Modified Paths

Removed Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (220534 => 220535)


--- trunk/LayoutTests/ChangeLog	2017-08-10 18:22:01 UTC (rev 220534)
+++ trunk/LayoutTests/ChangeLog	2017-08-10 18:27:01 UTC (rev 220535)
@@ -1,3 +1,18 @@
+2017-08-10  Commit Queue  <commit-qu...@webkit.org>
+
+        Unreviewed, rolling out r220463.
+        https://bugs.webkit.org/show_bug.cgi?id=175435
+
+        unaccepted fix (Requested by n_wang on #webkit).
+
+        Reverted changeset:
+
+        "AX: crash at
+        WebCore::AccessibilityObject::supportsARIALiveRegion() const +
+        24"
+        https://bugs.webkit.org/show_bug.cgi?id=175340
+        http://trac.webkit.org/changeset/220463
+
 2017-08-10  Ryan Haddad  <ryanhad...@apple.com>
 
         Update TestExpectations for imported/w3c/web-platform-tests/IndexedDB/interleaved-cursors.html.

Deleted: trunk/LayoutTests/accessibility/add-children-pseudo-element-expected.txt (220534 => 220535)


--- trunk/LayoutTests/accessibility/add-children-pseudo-element-expected.txt	2017-08-10 18:22:01 UTC (rev 220534)
+++ trunk/LayoutTests/accessibility/add-children-pseudo-element-expected.txt	2017-08-10 18:27:01 UTC (rev 220535)
@@ -1,12 +0,0 @@
-Language Email 
-Make sure that we are updating the render block flow element's children correctly.
-
-On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
-
-
-PASS element.childrenCount is 3
-PASS element.childrenCount is 2
-PASS successfullyParsed is true
-
-TEST COMPLETE
-

Deleted: trunk/LayoutTests/accessibility/add-children-pseudo-element.html (220534 => 220535)


--- trunk/LayoutTests/accessibility/add-children-pseudo-element.html	2017-08-10 18:22:01 UTC (rev 220534)
+++ trunk/LayoutTests/accessibility/add-children-pseudo-element.html	2017-08-10 18:27:01 UTC (rev 220535)
@@ -1,70 +0,0 @@
-<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML//EN">
-<html>
-<head>
-<script src=""
-</head>
-
-<style>
-.pseudo::after {
-  content: url(resources/svg-circle.svg);
-  width: 18px;
-  height: 20px;
-  position: absolute;
-  margin-top: 6px;
-  right: 6px
-}
-
-.pseudo.hidden::after {
-    content: ' ';
-}
-</style>
-
-<body id="body">
-
-<div id="container">
-<div id="test" style="float : left;">
-<span>Language</span>
-<input id="test">
-</div>
-
-<div id="float" style="display: inline-block;" class="pseudo">
-<span required="" style="color: rgb(194, 0, 0);">Email</span>
-<input type="text" required="" aria-required="true" value="" _onkeyup_="hidePseudo();">
-</div>
-
-</div>
-
-<p id="description"></p>
-<div id="console"></div>
-
-<script>
-
-    description("Make sure that we are updating the render block flow element's children correctly.");
-
-    if (window.accessibilityController) {
-        var element = accessibilityController.accessibleElementById("float");
-        shouldBe("element.childrenCount", "3");
-        
-        eventSender.keyDown('\t');
-        eventSender.keyDown('\t');
-        
-        shouldBe("element.childrenCount", "2");
-        showPseudo();
-        
-        function hidePseudo() {
-            document.getElementById("float").className += "hidden"
-        }
-        
-        function showPseudo() {
-            document.getElementById("float").className = "pseudo";
-        }
-                
-        // Make sure getting the attributes of its children won't cause crash
-        element.attributesOfChildren();
-    }
-
-</script>
-
-<script src=""
-</body>
-</html>

Deleted: trunk/LayoutTests/accessibility/resources/svg-circle.svg (220534 => 220535)


--- trunk/LayoutTests/accessibility/resources/svg-circle.svg	2017-08-10 18:22:01 UTC (rev 220534)
+++ trunk/LayoutTests/accessibility/resources/svg-circle.svg	2017-08-10 18:27:01 UTC (rev 220535)
@@ -1,6 +0,0 @@
-<svg xmlns="http://www.w3.org/2000/svg">
-     <g>
-        <text style="float:right"></text>
-        <text>circle</text>
-    </g>
-</svg>
\ No newline at end of file

Modified: trunk/Source/WebCore/ChangeLog (220534 => 220535)


--- trunk/Source/WebCore/ChangeLog	2017-08-10 18:22:01 UTC (rev 220534)
+++ trunk/Source/WebCore/ChangeLog	2017-08-10 18:27:01 UTC (rev 220535)
@@ -1,3 +1,18 @@
+2017-08-10  Commit Queue  <commit-qu...@webkit.org>
+
+        Unreviewed, rolling out r220463.
+        https://bugs.webkit.org/show_bug.cgi?id=175435
+
+        unaccepted fix (Requested by n_wang on #webkit).
+
+        Reverted changeset:
+
+        "AX: crash at
+        WebCore::AccessibilityObject::supportsARIALiveRegion() const +
+        24"
+        https://bugs.webkit.org/show_bug.cgi?id=175340
+        http://trac.webkit.org/changeset/220463
+
 2017-08-10  Michael Catanzaro  <mcatanz...@igalia.com>
 
         Remove ENABLE_GAMEPAD_DEPRECATED

Modified: trunk/Source/WebCore/rendering/RenderBlockFlow.cpp (220534 => 220535)


--- trunk/Source/WebCore/rendering/RenderBlockFlow.cpp	2017-08-10 18:22:01 UTC (rev 220534)
+++ trunk/Source/WebCore/rendering/RenderBlockFlow.cpp	2017-08-10 18:27:01 UTC (rev 220535)
@@ -24,7 +24,6 @@
 #include "config.h"
 #include "RenderBlockFlow.h"
 
-#include "AXObjectCache.h"
 #include "Editor.h"
 #include "FloatingObjects.h"
 #include "Frame.h"
@@ -2353,9 +2352,6 @@
     }
 
     setLogicalWidthForFloat(*floatingObject, logicalWidthForChild(floatBox) + marginStartForChild(floatBox) + marginEndForChild(floatBox));
-    
-    if (AXObjectCache* cache = document().existingAXObjectCache())
-        cache->childrenChanged(this);
 
     return m_floatingObjects->add(WTFMove(floatingObject));
 }
@@ -2393,9 +2389,6 @@
                 markLinesDirtyInBlockRange(0, logicalBottom);
             }
             m_floatingObjects->remove(&floatingObject);
-            
-            if (AXObjectCache* cache = document().existingAXObjectCache())
-                cache->childrenChanged(this);
         }
     }
 }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to