Changeset 212788 in webkit

Timestamp:

Feb 21, 2017 5:59:57 PM (7 years ago)

Author:

Ryan Haddad

Message:

Unreviewed, rolling out r212737.

This change caused an existing LayoutTest to crash.

Reverted changeset:

"REGRESSION(r207669): Crash after mutating selector text"
​https://bugs.webkit.org/show_bug.cgi?id=168655
​http://trac.webkit.org/changeset/212737

Location:

trunk

Files:

• LayoutTests/ChangeLog (modified) (1 diff)
• LayoutTests/fast/css/selector-text-mutation-crash-expected.txt (deleted)
• LayoutTests/fast/css/selector-text-mutation-crash.html (deleted)
• Source/WebCore/ChangeLog (modified) (1 diff)
• Source/WebCore/style/StyleScope.cpp (modified) (1 diff)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2017-02-21  Ryan Haddad  <ryanhaddad@apple.com>
+
+        Unreviewed, rolling out r212737.
+
+        This change caused an existing LayoutTest to crash.
+
+        Reverted changeset:
+
+        "REGRESSION(r207669): Crash after mutating selector text"
+        https://bugs.webkit.org/show_bug.cgi?id=168655
+        http://trac.webkit.org/changeset/212737
+
 2017-02-21  Chris Dumez  <cdumez@apple.com>
 

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2017-02-21  Ryan Haddad  <ryanhaddad@apple.com>
+
+        Unreviewed, rolling out r212737.
+
+        This change caused an existing LayoutTest to crash.
+
+        Reverted changeset:
+
+        "REGRESSION(r207669): Crash after mutating selector text"
+        https://bugs.webkit.org/show_bug.cgi?id=168655
+        http://trac.webkit.org/changeset/212737
+
 2017-02-21  Chris Dumez  <cdumez@apple.com>
 

trunk/Source/WebCore/style/StyleScope.cpp

@@ -522 +522 @@
 void Scope::scheduleUpdate(UpdateType update)
 {
-    if (update == UpdateType::ContentsOrInterpretation)
-        clearResolver();
-
     if (!m_pendingUpdate || *m_pendingUpdate < update) {
         m_pendingUpdate = update;