Bug 65873

Summary: Deprecate rebaseline-chromium-webkit-tests
Product: WebKit Reporter: Adam Barth <abarth>
Component: New BugsAssignee: Adam Barth <abarth>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, eric, jamesr, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Adam Barth 2011-08-08 13:32:50 PDT
Deprecate rebaseline-chromium-webkit-tests
Comment 1 Adam Barth 2011-08-08 13:35:17 PDT
Created attachment 103287 [details]
Patch
Comment 2 WebKit Review Bot 2011-08-08 14:38:10 PDT
Comment on attachment 103287 [details]
Patch

Clearing flags on attachment: 103287

Committed r92637: <http://trac.webkit.org/changeset/92637>
Comment 3 WebKit Review Bot 2011-08-08 14:38:14 PDT
All reviewed patches have been landed.  Closing bug.
Comment 4 James Robinson 2011-08-10 13:44:22 PDT
This patch adds a deprecation warning, but AFAIK webkit-patch rebaseline-expectations doesn't know about the chromium canaries, so there is a *significant* functionality loss.  I don't think that a chromium gardener today can actually use rebaseline-expectations to do their job so this warning is misleading at best.
Comment 5 Adam Barth 2011-08-10 13:52:12 PDT
Have you tried using the script?  It does know about the Chromium canaries.
Comment 6 James Robinson 2011-08-10 13:54:38 PDT
Woah, deep magic.  Never mind, then :)