Move Chromium's TestExpectations and ContentShellTestExpectations to generic location

Move Chromium's TestExpectations and ContentShellTestExpectations to generic location

BUG=230264
R=dpranke,jparent

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=148348

[Steve Block, 2013-04-11 12:10:20]

This depends on https://codereview.chromium.org/14047003

Adding jparent for loader.js

[Julie Parent, 2013-04-11 22:40:13]

https://codereview.chromium.org/14172003/diff/2001/Tools/TestResultServer/sta...
File Tools/TestResultServer/static-dashboards/loader.js (right):

https://codereview.chromium.org/14172003/diff/2001/Tools/TestResultServer/sta...
Tools/TestResultServer/static-dashboards/loader.js:36: var
CHROMIUM_EXPECTATIONS_URL =
'http://svn.webkit.org/repository/webkit/trunk/LayoutTests/TestExpectations';
The dashboards definitely need to be updated, but I think this line isn't it. 
The best I can tell, this variable is unused :(

[Steve Block, 2013-04-11 23:54:45]

https://codereview.chromium.org/14172003/diff/2001/Tools/TestResultServer/sta...
File Tools/TestResultServer/static-dashboards/loader.js (right):

https://codereview.chromium.org/14172003/diff/2001/Tools/TestResultServer/sta...
Tools/TestResultServer/static-dashboards/loader.js:36: var
CHROMIUM_EXPECTATIONS_URL =
'http://svn.webkit.org/repository/webkit/trunk/LayoutTests/TestExpectations';
OK, thanks. I'll remove my change here.

[abarth-chromium, 2013-04-12 02:07:26]

https://codereview.chromium.org/14172003/diff/8001/LayoutTests/platform/chrom...
File LayoutTests/platform/chromium-mac-mountainlion/TestExpectations (left):

https://codereview.chromium.org/14172003/diff/8001/LayoutTests/platform/chrom...
LayoutTests/platform/chromium-mac-mountainlion/TestExpectations:3: # passing
cleanly on Mountain Lion.
Maybe we should just delete this file?  It shouldn't be needed anymore.

[Dirk Pranke, 2013-04-12 02:12:00]

lgtm, except that you can't move ContentShellTestExpectations w/o updating the
bot to know about the new location, I think.

Please get jochen to approve this part and/or figure out what to do here. We
should probably be conditionally including this file in chromium.py at this
point, and not controlling this on the bots.

This means modifying the contentshell_latest_*_cfg.py files in
src/build/masters/master.chromium.webkit and restarting the master :(.

[Dirk Pranke, 2013-04-12 02:12:37]

On 2013/04/12 02:12:00, Dirk Pranke wrote:
> lgtm, except that you can't move ContentShellTestExpectations w/o updating the
> bot to know about the new location, I think.
> 
> Please get jochen to approve this part and/or figure out what to do here. We
> should probably be conditionally including this file in chromium.py at this
> point, and not controlling this on the bots.
> 
> This means modifying the contentshell_latest_*_cfg.py files in
> src/build/masters/master.chromium.webkit and restarting the master :(.

maybe just leave the ContentShell expectation where it is until jochen can get
rid of that file altogether?

[Steve Block, 2013-04-12 02:18:47]

> Maybe we should just delete this file? 
Will do so in a prerequisite change.

> maybe just leave the ContentShell expectation where it is until jochen can get
> rid of that file altogether?
OK, will do. Filed https://code.google.com/p/chromium/issues/detail?id=230711 to
track that.

[Steve Block, 2013-04-15 01:56:31]

Committed patchset #4 manually as r148348 (presubmit successful).