WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
89494
[Chromium] fast/canvas/webgl/gl-teximage.html needs rebaseline
https://bugs.webkit.org/show_bug.cgi?id=89494
Summary
[Chromium] fast/canvas/webgl/gl-teximage.html needs rebaseline
Jeff Timanus
Reported
2012-06-19 11:45:18 PDT
wkbug.com/86275 causes platform/chromium/virtual/gpu/fast/canvas/webgl/gl-teximage.html to fail with text differences. A new rebaseline is required.
Attachments
Patch
(1.24 KB, patch)
2012-08-07 12:21 PDT
,
Jeff Timanus
no flags
Details
Formatted Diff
Diff
Patch
(10.55 KB, patch)
2012-08-07 13:02 PDT
,
Jeff Timanus
no flags
Details
Formatted Diff
Diff
Patch
(7.04 KB, patch)
2012-08-08 13:21 PDT
,
Jeff Timanus
no flags
Details
Formatted Diff
Diff
Show Obsolete
(2)
View All
Add attachment
proposed patch, testcase, etc.
Jeff Timanus
Comment 1
2012-08-07 12:21:31 PDT
Created
attachment 156971
[details]
Patch EWS trial to capture current state of failing test output.
Jeff Timanus
Comment 2
2012-08-07 13:02:28 PDT
Created
attachment 156982
[details]
Patch
Jeff Timanus
Comment 3
2012-08-07 13:06:39 PDT
Comment on
attachment 156982
[details]
Patch PTAL. White-space modifications were a result of the webkit-patch tool.
Kenneth Russell
Comment 4
2012-08-08 02:07:07 PDT
gman, zmo, if one of you could please review this perhaps enne could marshal it through.
Adrienne Walker
Comment 5
2012-08-08 10:33:11 PDT
Comment on
attachment 156982
[details]
Patch In general, you don't need a review to land a rebaseline. You can just land with a comment about it being unreviewed. If you're asking for a review, I'd probably say that the test should be modified so that it behaves the same on all platforms rather than checking in a different expectation. The fact that the virtual/gpu directory prints 50% for every value but the checked in expectations says 49% or 51% (but still passes) seems like a wart in the test case that should be removed. Maybe it shouldn't print any percent if it's a passing value.
Jeff Timanus
Comment 6
2012-08-08 13:21:18 PDT
Created
attachment 157278
[details]
Patch
Jeff Timanus
Comment 7
2012-08-08 13:23:13 PDT
Comment on
attachment 157278
[details]
Patch This patch removed the output of the percentage of passing pixels in DRT runs, but not when running in Chrome.
Adrienne Walker
Comment 8
2012-08-08 14:16:38 PDT
Comment on
attachment 157278
[details]
Patch R=me. Thanks for that change.
WebKit Review Bot
Comment 9
2012-08-08 15:13:39 PDT
Comment on
attachment 157278
[details]
Patch Clearing flags on attachment: 157278 Committed
r125099
: <
http://trac.webkit.org/changeset/125099
>
WebKit Review Bot
Comment 10
2012-08-08 15:13:44 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