Changeset 164387 in webkit

Timestamp:

Feb 19, 2014 1:56:27 PM (10 years ago)

Author:

rniwa@webkit.org

Message:

isEditablePosition shouldn't trigger synchronous layout
​https://bugs.webkit.org/show_bug.cgi?id=129026

Reviewed by Brent Fulgham.

Just trigger style recalc instead.

• editing/htmlediting.cpp:

(WebCore::isEditablePosition):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• editing/htmlediting.cpp (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2014-02-19  Ryosuke Niwa  <rniwa@webkit.org>
+
+        isEditablePosition shouldn't trigger synchronous layout
+        https://bugs.webkit.org/show_bug.cgi?id=129026
+
+        Reviewed by Brent Fulgham.
+
+        Just trigger style recalc instead.
+
+        * editing/htmlediting.cpp:
+        (WebCore::isEditablePosition):
+
 2014-02-19  Beth Dakin  <bdakin@apple.com>
 

trunk/Source/WebCore/editing/htmlediting.cpp

@@ -147 +147 @@
         return false;
     if (updateStyle == UpdateStyle)
-        node->document().updateLayoutIgnorePendingStylesheets();
+        node->document().updateStyleIfNeeded();
     else
         ASSERT(updateStyle == DoNotUpdateStyle);
@@ -153 +153 @@
     if (node->renderer() && node->renderer()->isTable())
         node = node->parentNode();
-    
+
     return node->hasEditableStyle(editableType);
 }