Bug 81762 - TiledBackingStore: Create the first round of tiles synchronously after setting the visible rect.
Summary: TiledBackingStore: Create the first round of tiles synchronously after settin...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jocelyn Turcotte
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-21 04:57 PDT by Jocelyn Turcotte
Modified: 2012-03-21 10:18 PDT (History)
1 user (show)

See Also:

Attachments
Patch (1.79 KB, patch)
2012-03-21 05:02 PDT, Jocelyn Turcotte 		kenneth: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jocelyn Turcotte 2012-03-21 04:57:20 PDT 
TiledBackingStore: Create the first round of tiles synchronously after setting the visible rect.
Comment 1 Jocelyn Turcotte 2012-03-21 05:02:38 PDT 
Created attachment 133013 [details]
Patch
Comment 2 Kenneth Rohde Christiansen 2012-03-21 05:04:44 PDT 
Comment on attachment 133013 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=133013&action=review

> Source/WebCore/platform/graphics/TiledBackingStore.cpp:78
> -    startBackingStoreUpdateTimer();
> +    createTiles();

Are you sure we dont need any of the ifś inside the startBAckingStoreUpdateTimer?
Comment 3 Jocelyn Turcotte 2012-03-21 08:30:16 PDT 
(In reply to comment #2)
> Are you sure we dont need any of the ifś inside the startBAckingStoreUpdateTimer?

Yep, the same protections are already in createTiles().
Comment 4 Jocelyn Turcotte 2012-03-21 10:18:27 PDT 
Committed r111561: <http://trac.webkit.org/changeset/111561>