Title: [120637] trunk/Source/WebKit/chromium
- Revision
- 120637
- Author
- [email protected]
- Date
- 2012-06-18 16:03:31 -0700 (Mon, 18 Jun 2012)
Log Message
[Chromium] Ensure layout has happened before calling into hasNonEmptyBoundingBox()
Also update existing code to use updateLayoutIgnorePendingStylesheets()
https://bugs.webkit.org/show_bug.cgi?id=89258
Reviewed by James Robinson.
* src/WebNode.cpp:
(WebKit::WebNode::isFocusable): Use updateLayoutIgnorePendingStylesheets()
(WebKit::WebNode::hasNonEmptyBoundingBox): Force layout before checking the bounding box
Modified Paths
Diff
Modified: trunk/Source/WebKit/chromium/ChangeLog (120636 => 120637)
--- trunk/Source/WebKit/chromium/ChangeLog 2012-06-18 22:37:55 UTC (rev 120636)
+++ trunk/Source/WebKit/chromium/ChangeLog 2012-06-18 23:03:31 UTC (rev 120637)
@@ -1,3 +1,15 @@
+2012-06-18 Ilya Sherman <[email protected]>
+
+ [Chromium] Ensure layout has happened before calling into hasNonEmptyBoundingBox()
+ Also update existing code to use updateLayoutIgnorePendingStylesheets()
+ https://bugs.webkit.org/show_bug.cgi?id=89258
+
+ Reviewed by James Robinson.
+
+ * src/WebNode.cpp:
+ (WebKit::WebNode::isFocusable): Use updateLayoutIgnorePendingStylesheets()
+ (WebKit::WebNode::hasNonEmptyBoundingBox): Force layout before checking the bounding box
+
2012-06-18 Ian Vollick <[email protected]>
[chromium] Ensure that skipping frames during an accelerated animation doesn't cause starvation
Modified: trunk/Source/WebKit/chromium/src/WebNode.cpp (120636 => 120637)
--- trunk/Source/WebKit/chromium/src/WebNode.cpp 2012-06-18 22:37:55 UTC (rev 120636)
+++ trunk/Source/WebKit/chromium/src/WebNode.cpp 2012-06-18 23:03:31 UTC (rev 120637)
@@ -153,7 +153,7 @@
bool WebNode::isFocusable() const
{
- m_private->document()->updateLayout();
+ m_private->document()->updateLayoutIgnorePendingStylesheets();
return m_private->isFocusable();
}
@@ -215,6 +215,7 @@
bool WebNode::hasNonEmptyBoundingBox() const
{
+ m_private->document()->updateLayoutIgnorePendingStylesheets();
return m_private->hasNonEmptyBoundingBox();
}
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
