Title: [199990] releases/WebKitGTK/webkit-2.12/Source/WebCore
- Revision
- 199990
- Author
- [email protected]
- Date
- 2016-04-25 04:21:05 -0700 (Mon, 25 Apr 2016)
Log Message
Merge r199155 - FrameView::qualifiesAsVisuallyNonEmpty() returns false when loading a Google search results page before search results are loaded, even though the header is visible
https://bugs.webkit.org/show_bug.cgi?id=156339
<rdar://problem/24491381>
Reviewed by Andreas Kling.
Jeff's testing indicates lowering the document height threshold improves things visually during page loading.
* page/FrameView.cpp:
(WebCore::FrameView::qualifiesAsVisuallyNonEmpty):
Lower document height threshold to from 200 to 48 pixels.
Modified Paths
Diff
Modified: releases/WebKitGTK/webkit-2.12/Source/WebCore/ChangeLog (199989 => 199990)
--- releases/WebKitGTK/webkit-2.12/Source/WebCore/ChangeLog 2016-04-25 10:59:06 UTC (rev 199989)
+++ releases/WebKitGTK/webkit-2.12/Source/WebCore/ChangeLog 2016-04-25 11:21:05 UTC (rev 199990)
@@ -1,3 +1,20 @@
+2016-04-07 Antti Koivisto <[email protected]>
+
+ FrameView::qualifiesAsVisuallyNonEmpty() returns false when loading a Google search results page before search results are loaded, even though the header is visible
+ https://bugs.webkit.org/show_bug.cgi?id=156339
+ <rdar://problem/24491381>
+
+ Reviewed by Andreas Kling.
+
+ Patch by Jeff Miller.
+
+ Jeff's testing indicates lowering the document height threshold improves things visually during page loading.
+
+ * page/FrameView.cpp:
+ (WebCore::FrameView::qualifiesAsVisuallyNonEmpty):
+
+ Lower document height threshold to from 200 to 48 pixels.
+
2016-04-06 Jer Noble <[email protected]>
CRASH in AudioDestinationNode::render()
Modified: releases/WebKitGTK/webkit-2.12/Source/WebCore/page/FrameView.cpp (199989 => 199990)
--- releases/WebKitGTK/webkit-2.12/Source/WebCore/page/FrameView.cpp 2016-04-25 10:59:06 UTC (rev 199989)
+++ releases/WebKitGTK/webkit-2.12/Source/WebCore/page/FrameView.cpp 2016-04-25 11:21:05 UTC (rev 199990)
@@ -4216,7 +4216,8 @@
return true;
// Require the document to grow a bit.
- static const int documentHeightThreshold = 200;
+ // Using a value of 48 allows the header on Google's search page to render immediately before search results populate later.
+ static const int documentHeightThreshold = 48;
LayoutRect overflowRect = documentElement->renderBox()->layoutOverflowRect();
if (snappedIntRect(overflowRect).height() < documentHeightThreshold)
return false;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
