Title: [113100] trunk/Source/WebKit2
Revision
113100
Author
[email protected]
Date
2012-04-03 15:54:07 -0700 (Tue, 03 Apr 2012)

Log Message

[Qt][WK2] Remove #if !USE(TILED_BACKING_STORE) from WebFrameLoaderClient::transitionToCommittedForNewPage()
https://bugs.webkit.org/show_bug.cgi?id=83070

Patch by Zalan Bujtas <[email protected]> on 2012-04-03
Reviewed by Andreas Kling.

It is preventing m_frameHasCustomRepresentation to be set properly and not in sync with
WebFrameLoaderClient::transitionToCommittedFromCachedFrame()

* WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::transitionToCommittedForNewPage):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (113099 => 113100)


--- trunk/Source/WebKit2/ChangeLog	2012-04-03 22:51:44 UTC (rev 113099)
+++ trunk/Source/WebKit2/ChangeLog	2012-04-03 22:54:07 UTC (rev 113100)
@@ -1,3 +1,16 @@
+2012-04-03  Zalan Bujtas  <[email protected]>
+
+        [Qt][WK2] Remove #if !USE(TILED_BACKING_STORE) from WebFrameLoaderClient::transitionToCommittedForNewPage()
+        https://bugs.webkit.org/show_bug.cgi?id=83070
+
+        Reviewed by Andreas Kling.
+
+        It is preventing m_frameHasCustomRepresentation to be set properly and not in sync with
+        WebFrameLoaderClient::transitionToCommittedFromCachedFrame()
+
+        * WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
+        (WebKit::WebFrameLoaderClient::transitionToCommittedForNewPage):
+
 2012-04-03  Jer Noble  <[email protected]>
 
         ESC key in full screen does not result in webkitFullScreenChange event.

Modified: trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp (113099 => 113100)


--- trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp	2012-04-03 22:51:44 UTC (rev 113099)
+++ trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp	2012-04-03 22:54:07 UTC (rev 113100)
@@ -1167,10 +1167,8 @@
     bool isMainFrame = webPage->mainWebFrame() == m_frame;
     bool shouldUseFixedLayout = isMainFrame && webPage->useFixedLayout();
 
-#if !USE(TILED_BACKING_STORE)
     const ResourceResponse& response = m_frame->coreFrame()->loader()->documentLoader()->response();
     m_frameHasCustomRepresentation = isMainFrame && WebProcess::shared().shouldUseCustomRepresentationForResponse(response);
-#endif
 
     m_frame->coreFrame()->createView(webPage->size(), backgroundColor, /* transparent */ false, IntSize(), shouldUseFixedLayout);
     m_frame->coreFrame()->view()->setTransparent(!webPage->drawsBackground());
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to