s/app_shell_browsertests/extensions_browsertests/g

s/app_shell_browsertests/extensions_browsertests/g

We would like to use app_shell as the basis for all
content-browser integration tests within the
extensions layer.

This simply moves the test target up to the parent
//extensions path and renames it to
extensions_browsertests

BUG=388893
R=yoz@chromium.org

Committed: https://crrev.com/ea9df2af195b5b08909dc30e05bab95f6cfcd66f
Cr-Commit-Position: refs/heads/master@{#312669}

[Ken Rockot(use gerrit already), 2015-01-21 21:49:21]

Yoyo, could you please take a look?

[Yoyo Zhou, 2015-01-21 23:47:01]

LGTM

https://codereview.chromium.org/797023006/diff/1/extensions/extensions.gyp
File extensions/extensions.gyp (right):

https://codereview.chromium.org/797023006/diff/1/extensions/extensions.gyp#ne...
extensions/extensions.gyp:1369: 'shell/test/shell_apitest.cc',
We should think about whether shell/test is still the right place/name for
these.

https://codereview.chromium.org/797023006/diff/1/testing/buildbot/chromium.li...
File testing/buildbot/chromium.linux.json (right):

https://codereview.chromium.org/797023006/diff/1/testing/buildbot/chromium.li...
testing/buildbot/chromium.linux.json:16: "extensions_browsertests",
nit: I think these should be kept in alphabetical order if they are already
somewhat so. (The previous file isn't, for example.)

[Ken Rockot(use gerrit already), 2015-01-22 00:22:48]

https://codereview.chromium.org/797023006/diff/1/extensions/extensions.gyp
File extensions/extensions.gyp (right):

https://codereview.chromium.org/797023006/diff/1/extensions/extensions.gyp#ne...
extensions/extensions.gyp:1369: 'shell/test/shell_apitest.cc',
On 2015/01/21 23:47:01, Yoyo Zhou wrote:
> We should think about whether shell/test is still the right place/name for
> these.

Yeah, I hesitated to leave this as-is, but the lines are sufficiently blurry
that I think it's OK to defer for now.

https://codereview.chromium.org/797023006/diff/1/testing/buildbot/chromium.li...
File testing/buildbot/chromium.linux.json (right):

https://codereview.chromium.org/797023006/diff/1/testing/buildbot/chromium.li...
testing/buildbot/chromium.linux.json:16: "extensions_browsertests",
On 2015/01/21 23:47:01, Yoyo Zhou wrote:
> nit: I think these should be kept in alphabetical order if they are already
> somewhat so. (The previous file isn't, for example.)

Done.

[Ken Rockot(use gerrit already), 2015-01-22 06:25:12]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2015-01-22 06:25:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/797023006/140001

[commit-bot: I haz the power, 2015-01-22 06:31:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-01-22 06:31:54]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Ken Rockot(use gerrit already), 2015-01-22 15:08:12]

rockot@chromium.org changed reviewers:
+ jochen@chromium.org

[Ken Rockot(use gerrit already), 2015-01-22 15:08:35]

jochen@ could you please take a look for build and testing/buildbot? Thanks!

[jochen (gone - plz use gerrit), 2015-01-22 16:25:49]

lgtm

would you mind swarming those tests btw?

[Ken Rockot(use gerrit already), 2015-01-22 16:40:46]

On 2015/01/22 16:25:49, jochen (slow) wrote:
> lgtm
> 
> would you mind swarming those tests btw?

sure, i'll do that in a follow-up CL. thanks!

[Ken Rockot(use gerrit already), 2015-01-22 16:40:59]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2015-01-22 16:41:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/797023006/140001

[commit-bot: I haz the power, 2015-01-22 16:42:31]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-01-22 16:42:32]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Ken Rockot(use gerrit already), 2015-01-22 18:26:55]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2015-01-22 18:29:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/797023006/140001

[commit-bot: I haz the power, 2015-01-22 18:48:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-01-22 18:48:33]

Try jobs failed on following builders:
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)
  linux_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu_trigger...)

[Ken Rockot(use gerrit already), 2015-01-22 20:24:34]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2015-01-22 20:25:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/797023006/140001

[commit-bot: I haz the power, 2015-01-22 20:27:03]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2015-01-22 20:29:02]

Patchset 8 (id:??) landed as
https://crrev.com/ea9df2af195b5b08909dc30e05bab95f6cfcd66f
Cr-Commit-Position: refs/heads/master@{#312669}