Summary: | Pages that use hardware acceleration don't repaint after waking computer from sleep | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Adam Roben (:aroben) <aroben> | ||||
Component: | Layout and Rendering | Assignee: | Adam Roben (:aroben) <aroben> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | cmarrin, enrica, eric, simon.fraser | ||||
Priority: | P2 | Keywords: | InRadar, PlatformOnly | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | Windows XP | ||||||
URL: | http://webkit.org/blog-files/3d-transforms/poster-circle.html | ||||||
Bug Depends on: | 39159 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Adam Roben (:aroben)
2010-05-14 14:51:38 PDT
Created attachment 56151 [details]
Periodically try to reset a lost IDirect3DDevice9 until we succeed
Comment on attachment 56151 [details] Periodically try to reset a lost IDirect3DDevice9 until we succeed > - resetDevice(); > + resetDevice(ChangedWindowSize); Is it worth asserting that this call returns true? r+ Attachment 56151 [details] was posted by a committer and has review+, assigning to Adam Roben for commit.
(In reply to comment #3) > (From update of attachment 56151 [details]) > > - resetDevice(); > > + resetDevice(ChangedWindowSize); > > Is it worth asserting that this call returns true? No. Resetting might fail at this point (e.g., if the device became lost sometime since the last time we rendered), so we can't assert that it doesn't. But we don't need to worry here whether it failed or not; if it did fail, we'll try to reset again before we next render. I'll add a comment about this. Thanks for the review! Committed r59610: <http://trac.webkit.org/changeset/59610> |