Stop building Chrome Frame tests.

Stop building Chrome Frame tests.

They are no longer run on any bots, so save time by not building them.

BUG=316496
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=242796

[grt (UTC plus 2), 2013-12-17 20:28:53]

Can either of you think of any other configuration that needs to be adjusted for
this change? I wouldn't want anything on the waterfall to break when these tests
suddenly aren't built anymore. Thanks.

[M-A Ruel, 2013-12-17 20:43:49]

Before committing this, the chrome_frame steps must have been removed from the
masters and the masters restarted. I still see some chrome_frame references
under https://chromium.googlesource.com/a/chromium/tools/build.git.

I assume you still want to keep the code for a while?

https://codereview.chromium.org/100213009/diff/1/build/all.gyp
File build/all.gyp (right):

https://codereview.chromium.org/100213009/diff/1/build/all.gyp#newcode156
build/all.gyp:156: # TODO(jschuh) Enable Win64 Chrome Frame. crbug.com/176875
I think crbug.com/176875 can be marked as WontFix.

https://codereview.chromium.org/100213009/diff/1/build/all.gyp#newcode157
build/all.gyp:157: '../chrome_frame/chrome_frame.gyp:npchrome_frame',
What's npchrome_frame, why is it needed?

[grt (UTC plus 2), 2013-12-18 04:10:36]

On 2013/12/17 20:43:49, M-A Ruel wrote:
> Before committing this, the chrome_frame steps must have been removed from the
> masters and the masters restarted. I still see some chrome_frame references
> under https://chromium.googlesource.com/a/chromium/tools/build.git.

Thanks for the pointer. See https://codereview.chromium.org/110373008/ for a CL
to remove that stuff.

> I assume you still want to keep the code for a while?

I'm in the process of removing the Chrome Frame code.

[grt (UTC plus 2), 2013-12-18 04:10:48]

https://codereview.chromium.org/100213009/diff/1/build/all.gyp
File build/all.gyp (right):

https://codereview.chromium.org/100213009/diff/1/build/all.gyp#newcode156
build/all.gyp:156: # TODO(jschuh) Enable Win64 Chrome Frame. crbug.com/176875
On 2013/12/17 20:43:49, M-A Ruel wrote:
> I think crbug.com/176875 can be marked as WontFix.

Done.

https://codereview.chromium.org/100213009/diff/1/build/all.gyp#newcode157
build/all.gyp:157: '../chrome_frame/chrome_frame.gyp:npchrome_frame',
On 2013/12/17 20:43:49, M-A Ruel wrote:
> What's npchrome_frame, why is it needed?

npchrome_frame is the target for the shipping binary. i will remove this
shortly. the change at hand is just to get rid of the weight of the tests.

[grt (UTC plus 2), 2013-12-23 18:45:17]

The buildbot config changes have landed. PTAL.

[M-A Ruel, 2013-12-23 19:04:04]

lgtm

[commit-bot: I haz the power, 2013-12-23 20:52:37]

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

[grt (UTC plus 2), 2013-12-23 21:05:42]

+jam for chrome/chrome.gyp OWNERS review.

[commit-bot: I haz the power, 2013-12-23 21:12:13]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[jam, 2013-12-24 00:09:10]

On 2013/12/23 21:05:42, grt wrote:
> +jam for chrome/chrome.gyp OWNERS review.

rubberstamp lgtm. feel free to TBR these changes

[commit-bot: I haz the power, 2014-01-02 15:20:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/grt@chromium.org/100213009/170001

[commit-bot: I haz the power, 2014-01-02 20:01:25]

Retried try job too often on win_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2014-01-02 20:30:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/grt@chromium.org/100213009/170001

[commit-bot: I haz the power, 2014-01-02 21:13:44]

Change committed as 242796