Bug 115172

Summary: REGRESSION (r147797): Animations slideshows of images on www.thesuperficial.com are slow
Product: WebKit Reporter: Antti Koivisto <koivisto>
Component: Page LoadingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, esprehn+autocc, japhet, kling, simon.fraser
Priority: P2 Keywords: InRadar, Regression
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
URL: http://www.thesuperficial.com/photos/reese-witherspoon-at-a-screening-of-mud
Attachments:
Description Flags
patch kling: review+

Antti Koivisto
Reported 2013-04-25 05:49:09 PDT
On this page ad subframes loads on slideshow navigation switch us to throttled state. We should only throttle layer flushes for the main load, not for any subsequent dynamically triggered subframe load.
Attachments
patch (4.09 KB, patch)
2013-04-25 05:58 PDT, Antti Koivisto 		kling: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Antti Koivisto
Comment 1 2013-04-25 05:58:18 PDT
Created attachment 199655 [details] patch
Antti Koivisto
Comment 2 2013-04-25 05:58:49 PDT
<rdar://problem/13724574>
Andreas Kling
Comment 3 2013-04-25 06:01:54 PDT
Comment on attachment 199655 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=199655&action=review r=me > Source/WebCore/loader/FrameLoader.cpp:3322 > - bool isLoadProgressing = m_frame->page()->progress()->isLoadProgressing(); > + bool isLoadProgressing = m_frame->page()->progress()->isMainLoadProgressing(); You may also want to rename the variable.
Antti Koivisto
Comment 4 2013-04-25 06:25:13 PDT
http://trac.webkit.org/changeset/149105
Note You need to log in before you can comment on or make changes to this bug.