WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
54625
[chromium] GradientSkia: use the common Gradient stop sorting methods.
https://bugs.webkit.org/show_bug.cgi?id=54625
Summary
[chromium] GradientSkia: use the common Gradient stop sorting methods.
noel gordon
Reported
2011-02-17 01:22:58 PST
Address a code FIXME.
Attachments
Patch
(1.82 KB, patch)
2011-02-17 01:27 PST
,
noel gordon
no flags
Details
Formatted Diff
Diff
patch - merge and reupload
(1.79 KB, patch)
2011-02-27 21:48 PST
,
noel gordon
no flags
Details
Formatted Diff
Diff
patch - merge and reupload #2
(1.79 KB, patch)
2011-02-27 21:55 PST
,
noel gordon
no flags
Details
Formatted Diff
Diff
Show Obsolete
(2)
View All
Add attachment
proposed patch, testcase, etc.
noel gordon
Comment 1
2011-02-17 01:27:39 PST
Created
attachment 82768
[details]
Patch
James Robinson
Comment 2
2011-02-17 11:04:27 PST
Comment on
attachment 82768
[details]
Patch R=me
WebKit Commit Bot
Comment 3
2011-02-18 18:32:26 PST
Comment on
attachment 82768
[details]
Patch Rejecting
attachment 82768
[details]
from commit-queue. Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=cr-jail-3', 'land-a..." exit_code: 2 Last 500 characters of output: c (refs/remotes/trunk) M Source/WebCore/ChangeLog M Source/WebCore/platform/graphics/skia/GradientSkia.cpp
r79063
= 29f47ae159022f5b8d71afc2ca3b29c700d3739a (refs/remotes/trunk) M Source/WebKit2/ChangeLog M Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp M Source/WebKit2/WebProcess/WebPage/WebPage.cpp
r79064
= 8c69a75bf9b2ef43953139bbda1848f2a4ca9f84 (refs/remotes/trunk) First, rewinding head to replay your work on top of it... Fast-forwarded master to refs/remotes/trunk. Full output:
http://queues.webkit.org/results/7939200
James Robinson
Comment 4
2011-02-18 18:47:48 PST
Looks like this doesn't apply to ToT any more. Would you mind merging+uploading, Noel?
WebKit Commit Bot
Comment 5
2011-02-18 19:01:56 PST
The commit-queue encountered the following flaky tests while processing
attachment 82768
[details]
: http/tests/security/cross-frame-access-port-explicit-domain.html
bug 54668
(author:
sam@webkit.org
) The commit-queue is continuing to process your patch.
noel gordon
Comment 6
2011-02-19 16:54:07 PST
(In reply to
comment #4
)
> Looks like this doesn't apply to ToT any more. Would you mind merging+uploading, Noel?
No worries. I'm away for a week, and will merge+upload on my return.
noel gordon
Comment 7
2011-02-27 21:48:56 PST
Created
attachment 84015
[details]
patch - merge and reupload
WebKit Review Bot
Comment 8
2011-02-27 21:51:26 PST
Attachment 84015
[details]
did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/platform/graphics/skia/GradientSkia.cpp:118: One or more unexpected \r (^M) found; better to use only a \n [whitespace/carriage_return] [1] Suppressing further [whitespace/carriage_return] reports for this file. Total errors found: 1 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.
noel gordon
Comment 9
2011-02-27 21:55:18 PST
Created
attachment 84016
[details]
patch - merge and reupload #2
WebKit Commit Bot
Comment 10
2011-02-28 17:31:14 PST
The commit-queue encountered the following flaky tests while processing
attachment 84016
[details]
: http/tests/websocket/tests/close-on-navigate-new-location.html
bug 55431
(author:
abarth@webkit.org
) The commit-queue is continuing to process your patch.
WebKit Commit Bot
Comment 11
2011-02-28 17:34:07 PST
Comment on
attachment 84016
[details]
patch - merge and reupload #2 Clearing flags on attachment: 84016 Committed
r79950
: <
http://trac.webkit.org/changeset/79950
>
WebKit Commit Bot
Comment 12
2011-02-28 17:34:11 PST
All reviewed patches have been landed. Closing bug.
WebKit Review Bot
Comment 13
2011-02-28 18:03:37 PST
http://trac.webkit.org/changeset/79950
might have broken SnowLeopard Intel Release (Build)
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