Bug 71833 - [chromium] remove the deprecated rebaseline script
Summary: [chromium] remove the deprecated rebaseline script
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Tony Chang
Depends on:
Reported: 2011-11-08 10:32 PST by Tony Chang
Modified: 2012-06-28 10:11 PDT (History)
6 users (show)

See Also:

Patch (69.61 KB, patch)
2011-11-08 10:33 PST, Tony Chang
eric: review+
rniwa: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tony Chang 2011-11-08 10:32:48 PST
[chromium] remove the deprecated rebaseline script
Comment 1 Tony Chang 2011-11-08 10:33:05 PST
Created attachment 114112 [details]
Comment 2 Eric Seidel (no email) 2011-11-08 11:44:50 PST
Comment on attachment 114112 [details]

Comment 3 Ryosuke Niwa 2011-11-08 12:03:58 PST
Comment on attachment 114112 [details]

Please don't remove this tool. I actively use it.
Comment 4 Ryosuke Niwa 2011-11-08 12:04:22 PST
New tool is practically unusable for me for the reasons stated in the webkit-dev thread.
Comment 5 Dirk Pranke 2011-11-08 12:27:29 PST
FWIW, I also don't find webkit-patch rebaseline a complete replacement for rebaseline-chromium-...

Things I miss:

1) visual diff of what's changed. rebaseline-chromium's visual diff was buggy, but was useful when it worked. If we could make webkit-patch pretty-diff work, that would be good enough.

2) The last time I looked, it was pretty much impossible to figure out what webkit-patch rebaseline actually did, and why. I had spent a fair amount of time cleaning up the logging in rebaseline-chromium so that you could tell what did get updated and why. In addition, combining the "update baselines" step with the "optimize baselines" step makes it very difficult to figure out what is actually going on.

I don't want to stop us from getting rid of old code, so I would be fine w/ obsoleting this now as long as someone was actively working on these tools/enhancements in the near future. Saying "patches welcome" is not good enough ;)
Comment 6 Tony Chang 2011-11-08 12:32:52 PST
FWIW, I don't think it's important to understand why certain files are in certain directories.  The tool should do the right thing and we shouldn't require people to understand it.

Having a visual diff for a specific platform seems useful though.

We should probably just fix the flakiness dashboard or garden-o-matic to do this instead of just barfing up a bunch of iframes.
Comment 7 Ojan Vafai 2012-06-28 10:11:06 PDT
Looks like this script was removed at some point.