WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Details
Formatted Diff
Diff
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
Details
View All
Add attachment
proposed patch, testcase, etc.
Jae Hyun Park
Comment 1
2013-06-09 17:51:33 PDT
Created
attachment 204128
[details]
Patch
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.
Top of Page
Format For Printing
XML
Clone This Bug