Bug 170836 - The Incomplete asynchronously decoded image frame should be decoded every time it's drawn
Summary: The Incomplete asynchronously decoded image frame should be decoded every tim...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Images (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Said Abou-Hallawa
URL:
Keywords: InRadar
Depends on:
Blocks: 172009
  Show dependency tree / graph
 
Reported: 2017-04-13 16:53 PDT by Said Abou-Hallawa
Modified: 2017-05-11 19:34 PDT (History)
3 users (show)

See Also:

Attachments
Patch (3.42 KB, patch)
2017-04-13 16:55 PDT, Said Abou-Hallawa 		no flags Details | Formatted Diff | Diff
Patch (5.95 KB, patch)
2017-04-14 10:38 PDT, Said Abou-Hallawa 		no flags Details | Formatted Diff | Diff
Patch (5.63 KB, patch)
2017-05-08 16:59 PDT, Said Abou-Hallawa 		no flags Details | Formatted Diff | Diff
Show Obsolete (2) 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 Said Abou-Hallawa 2017-04-13 16:53:56 PDT 
The asynchronously decoded image frames has to be cached to prevent flickering, but we have to keep requesting new decoding for the incomplete frame every time it's drawn. This is to avoid drawing an incomplete image frame even after all its encoded data is received.
Comment 1 Said Abou-Hallawa 2017-04-13 16:55:22 PDT 
Created attachment 307048 [details]
Patch
Comment 2 Said Abou-Hallawa 2017-04-13 16:56:18 PDT 
<rdar://problem/31132092>
Comment 3 Said Abou-Hallawa 2017-04-14 10:38:40 PDT 
Created attachment 307122 [details]
Patch
Comment 4 Tim Horton 2017-04-17 10:27:35 PDT 
Can we test this?
Comment 5 Said Abou-Hallawa 2017-05-08 16:59:58 PDT 
Created attachment 309441 [details]
Patch
Comment 6 Said Abou-Hallawa 2017-05-08 18:53:56 PDT 
Comment on attachment 309441 [details]
Patch

Clearing flags on attachment: 309441

Committed r216471: <http://trac.webkit.org/changeset/216471>
Comment 7 Said Abou-Hallawa 2017-05-08 18:53:57 PDT 
All reviewed patches have been landed.  Closing bug.