RESOLVED FIXED Bug 117384
[CoordinatedGraphics] Typo in argument decoder for CoordinatedGraphicsState
https://bugs.webkit.org/show_bug.cgi?id=117384
Summary [CoordinatedGraphics] Typo in argument decoder for CoordinatedGraphicsState
Ralph T
Reported 2013-06-09 12:04:08 PDT
I noticed that CoordinatedGraphicsArgumentCoders.cpp:1187 is wrong for CSS shaders, it currently reads: if (!decoder.decode(state.updateAtlasesToRemove)) return false; But it should say: if (!decoder.decode(state.customFiltersToRemove)) return false;
Attachments
Patch (1.61 KB, patch)
2013-06-09 17:51 PDT, Jae Hyun Park
no flags
Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2 (631.13 KB, application/zip)
2013-06-09 18:42 PDT, Build Bot
no flags
Jae Hyun Park
Comment 1 2013-06-09 17:51:33 PDT
Jae Hyun Park
Comment 2 2013-06-09 17:53:14 PDT
(In reply to comment #0) > I noticed that CoordinatedGraphicsArgumentCoders.cpp:1187 is wrong for CSS shaders, it currently reads: > > if (!decoder.decode(state.updateAtlasesToRemove)) > return false; > > But it should say: > > if (!decoder.decode(state.customFiltersToRemove)) > return false; Thanks for reporting! I quickly made a patch to fix this. I hope you don't mind.
Build Bot
Comment 3 2013-06-09 18:42:34 PDT
Comment on attachment 204128 [details] Patch Attachment 204128 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/803133 New failing tests: http/tests/security/cross-origin-plugin-private-browsing-toggled.html
Build Bot
Comment 4 2013-06-09 18:42:36 PDT
Created attachment 204131 [details] Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-12 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.3
Ralph T
Comment 5 2013-06-09 21:29:14 PDT
(In reply to comment #2) > Thanks for reporting! > I quickly made a patch to fix this. I hope you don't mind. Not at all, thanks for being proactive :).
Noam Rosenthal
Comment 6 2013-06-09 23:05:08 PDT
Comment on attachment 204128 [details] Patch EWS responds to a flaky test, this code does not get compiled on Mac.
WebKit Commit Bot
Comment 7 2013-06-09 23:25:54 PDT
Comment on attachment 204128 [details] Patch Clearing flags on attachment: 204128 Committed r151364: <http://trac.webkit.org/changeset/151364>
WebKit Commit Bot
Comment 8 2013-06-09 23:25:57 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.