Bug 36396 - disable chromium's downstream expectations and baselines temporarily for testing
Summary: disable chromium's downstream expectations and baselines temporarily for testing
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Dirk Pranke
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-03-19 16:17 PDT by Dirk Pranke
Modified: 2010-03-19 17:41 PDT (History)
2 users (show)

See Also:


Attachments
Patch (5.32 KB, patch)
2010-03-19 16:18 PDT, Dirk Pranke
dglazkov: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Pranke 2010-03-19 16:17:24 PDT
We need to temporarily not look in the downstream chromium.org repo for test expectations or baselines to make sure the upstream version is working correctly.
Comment 1 Dirk Pranke 2010-03-19 16:18:58 PDT
Created attachment 51202 [details]
Patch
Comment 2 Dimitri Glazkov (Google) 2010-03-19 16:20:18 PDT
Comment on attachment 51202 [details]
Patch

beam me up, Scotty.
Comment 3 Ojan Vafai 2010-03-19 16:24:05 PDT
Why temporarily? We should just do this permanently.
Comment 4 Dirk Pranke 2010-03-19 16:46:38 PDT
(In reply to comment #3)
> Why temporarily? We should just do this permanently.

Well, we do actually want to keep the overrides file around. We can probably delete the downstream baseline paths permanently, but I'll at least leave the paths commented out until the directories have actually been removed.
Comment 5 Ojan Vafai 2010-03-19 16:51:21 PDT
(In reply to comment #4)
> (In reply to comment #3)
> > Why temporarily? We should just do this permanently.
> 
> Well, we do actually want to keep the overrides file around. We can probably
> delete the downstream baseline paths permanently, but I'll at least leave the
> paths commented out until the directories have actually been removed.

The FIXME's imply that we won't remove the downstream directories.

I don't see the point in leaving in commented out code. If we want to add them back, the old lines are still in our revision history.
Comment 6 Ojan Vafai 2010-03-19 16:52:08 PDT
Also, can't we just make the overrides file an empty text file, then we wouldn't need to comment out this code?
Comment 7 Dirk Pranke 2010-03-19 17:15:53 PDT
Fair enough. I'll delete the lines for now.(In reply to comment #5)
> (In reply to comment #4)
> > (In reply to comment #3)
> > > Why temporarily? We should just do this permanently.
> > 
> > Well, we do actually want to keep the overrides file around. We can probably
> > delete the downstream baseline paths permanently, but I'll at least leave the
> > paths commented out until the directories have actually been removed.
> 
> The FIXME's imply that we won't remove the downstream directories.
> 
> I don't see the point in leaving in commented out code. If we want to add them
> back, the old lines are still in our revision history.

Fair enough. I'll delete the lines for now.
Comment 8 Dirk Pranke 2010-03-19 17:18:04 PDT
(In reply to comment #6)
> Also, can't we just make the overrides file an empty text file, then we
> wouldn't need to comment out this code?

No, since the file is in the chromium repo. Once we're content that the tests pass just looking at the webkit directories and the webkit expectations, we'll add the reference to the downstream file back in, and then when we do the next webkit roll, we'll empty out the file (and tell people what the process for updating it is).
Comment 9 Dirk Pranke 2010-03-19 17:19:04 PDT
Note that, for the record, my confidence that we can tell people to simply put baselines upstream is pretty low. I bet people are going to want to be able to check in baselines downstream ...
Comment 10 Dirk Pranke 2010-03-19 17:41:37 PDT
Committed r56287: <http://trac.webkit.org/changeset/56287>