Add files to unit_tests.isolate and browser_tests.isolate.

Add files to unit_tests.isolate and browser_tests.isolate.

R=thakis@chromium.org
BUG=98636
TEST=


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=140220

[M-A Ruel, 2012-06-02 02:12:33]

Add a few other files to the dependencies. browser_tests pulls a lot of
python_26 files that is going to be trimmed off eventually as a separate change.
We're living with it for now.

[Nico, 2012-06-02 02:17:01]

lgtm stamp

https://chromiumcodereview.appspot.com/10505006/diff/1/chrome/browser_tests.i...
File chrome/browser_tests.isolate (right):

https://chromiumcodereview.appspot.com/10505006/diff/1/chrome/browser_tests.i...
chrome/browser_tests.isolate:23: '<(PRODUCT_DIR)/chrome<(EXECUTABLE_SUFFIX)',
This file doesn't exist on mac.

https://chromiumcodereview.appspot.com/10505006/diff/1/chrome/browser_tests.i...
chrome/browser_tests.isolate:42: '<(PRODUCT_DIR)/Chromium
Framework.framework/Versions/A/resources/BookmarkBar.nib',
Is the plan to keep these lists updated by hand? :-/

[M-A Ruel, 2012-06-03 00:51:48]

https://chromiumcodereview.appspot.com/10505006/diff/1/chrome/browser_tests.i...
File chrome/browser_tests.isolate (right):

https://chromiumcodereview.appspot.com/10505006/diff/1/chrome/browser_tests.i...
chrome/browser_tests.isolate:23: '<(PRODUCT_DIR)/chrome<(EXECUTABLE_SUFFIX)',
On 2012/06/02 02:17:02, Nico wrote:
> This file doesn't exist on mac.

This is the 'linux' section.

https://chromiumcodereview.appspot.com/10505006/diff/1/chrome/browser_tests.i...
chrome/browser_tests.isolate:42: '<(PRODUCT_DIR)/Chromium
Framework.framework/Versions/A/resources/BookmarkBar.nib',
On 2012/06/02 02:17:02, Nico wrote:
> Is the plan to keep these lists updated by hand? :-/

No. This list was semi-automatically generated, I had to replace the
<(PRODUCT_DIR) section manually, added <(EXECUTABLE_SUFFIX) and remove the .log
files. The lists themselves and the conditions were generated automatically.

Tracing browser_tests is more challenging than expected so I only traced an
~hundred test cases.

[commit-bot: I haz the power, 2012-06-03 00:51:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/maruel@chromium.org/10505006/1

[commit-bot: I haz the power, 2012-06-03 00:52:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/maruel@chromium.org/10505006/1

[Nico, 2012-06-03 00:52:42]

https://chromiumcodereview.appspot.com/10505006/diff/1/chrome/browser_tests.i...
File chrome/browser_tests.isolate (right):

https://chromiumcodereview.appspot.com/10505006/diff/1/chrome/browser_tests.i...
chrome/browser_tests.isolate:23: '<(PRODUCT_DIR)/chrome<(EXECUTABLE_SUFFIX)',
On 2012/06/03 00:51:49, Marc-Antoine Ruel wrote:
> On 2012/06/02 02:17:02, Nico wrote:
> > This file doesn't exist on mac.
> 
> This is the 'linux' section.

Hm, I actually noticed that and thought I had deleted this comment before
hitting 'publish'. Sorry!

[commit-bot: I haz the power, 2012-06-03 03:19:29]

Change committed as 140220

[rniwa-cr, 2012-06-03 05:00:06]

It could be that this CL broke gclient sync on Windows:

http://build.chromium.org/p/chromium/buildstatus?builder=Win&number=11422
http://build.chromium.org/p/chromium/buildstatus?builder=Win%20Builder&number...
http://build.chromium.org/p/chromium/buildstatus?builder=Win%20Builder%202010...
http://build.chromium.org/p/chromium/buildstatus?builder=Win%20Builder%20%28d...
http://build.chromium.org/p/chromium/buildstatus?builder=Win%20Aura&number=14116

[rniwa-cr, 2012-06-03 05:24:09]

Reverted the patch in
http://src.chromium.org/viewvc/chrome?view=rev&revision=140221 since clobbering
the bot didn't fix the problem.