Title: [115101] trunk/Source/WebCore
- Revision
- 115101
- Author
- [email protected]
- Date
- 2012-04-24 13:15:02 -0700 (Tue, 24 Apr 2012)
Log Message
Update the scroll layer position on the main thread when viewing full-frame images
https://bugs.webkit.org/show_bug.cgi?id=84754
<rdar://problem/11193304>
Reviewed by Andreas Kling.
Showing really big images can be slow sometimes and in that case it's better to update the scroll layer position on
the main thread, which will avoid flashing white.
* page/scrolling/ScrollingCoordinator.cpp:
(WebCore::ScrollingCoordinator::updateShouldUpdateScrollLayerPositionOnMainThread):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (115100 => 115101)
--- trunk/Source/WebCore/ChangeLog 2012-04-24 20:01:45 UTC (rev 115100)
+++ trunk/Source/WebCore/ChangeLog 2012-04-24 20:15:02 UTC (rev 115101)
@@ -1,3 +1,17 @@
+2012-04-24 Anders Carlsson <[email protected]>
+
+ Update the scroll layer position on the main thread when viewing full-frame images
+ https://bugs.webkit.org/show_bug.cgi?id=84754
+ <rdar://problem/11193304>
+
+ Reviewed by Andreas Kling.
+
+ Showing really big images can be slow sometimes and in that case it's better to update the scroll layer position on
+ the main thread, which will avoid flashing white.
+
+ * page/scrolling/ScrollingCoordinator.cpp:
+ (WebCore::ScrollingCoordinator::updateShouldUpdateScrollLayerPositionOnMainThread):
+
2012-04-24 Caio Marcelo de Oliveira Filho <[email protected]>
Use Vector<Attribute> directly instead of encapsulating it in AttributeVector
Modified: trunk/Source/WebCore/page/scrolling/ScrollingCoordinator.cpp (115100 => 115101)
--- trunk/Source/WebCore/page/scrolling/ScrollingCoordinator.cpp 2012-04-24 20:01:45 UTC (rev 115100)
+++ trunk/Source/WebCore/page/scrolling/ScrollingCoordinator.cpp 2012-04-24 20:15:02 UTC (rev 115101)
@@ -327,7 +327,7 @@
FrameView* frameView = m_page->mainFrame()->view();
// FIXME: Having fixed objects on the page should not trigger the slow path.
- setShouldUpdateScrollLayerPositionOnMainThread(m_forceMainThreadScrollLayerPositionUpdates || frameView->hasSlowRepaintObjects() || frameView->hasFixedObjects());
+ setShouldUpdateScrollLayerPositionOnMainThread(m_forceMainThreadScrollLayerPositionUpdates || frameView->hasSlowRepaintObjects() || frameView->hasFixedObjects() || m_page->mainFrame()->document()->isImageDocument());
}
void ScrollingCoordinator::setForceMainThreadScrollLayerPositionUpdates(bool forceMainThreadScrollLayerPositionUpdates)
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
