[Android] Move testing/android/ to testing/android/native_test/

[Android] Move testing/android/ to testing/android/native_test/

This will let us put android test code unrelated to native test APK wrapping
elsewhere in testing/android/*.

BUG=

Committed: https://crrev.com/6ab22495db53ab1a423e90f9f698c36a92f9eb62
Cr-Commit-Position: refs/heads/master@{#324512}

[jbudorick, 2015-04-08 15:56:49]

jbudorick@chromium.org changed reviewers:
+ tedchoc@chromium.org, yfriedman@chromium.org

[jbudorick, 2015-04-08 15:56:49]

split from https://codereview.chromium.org/1034053002/

[jbudorick, 2015-04-08 16:01:02]

https://codereview.chromium.org/1071573003/diff/1/testing/android/native_test...
File testing/android/native_test.gyp (right):

https://codereview.chromium.org/1071573003/diff/1/testing/android/native_test...
testing/android/native_test.gyp:1: # Copyright (c) 2012 The Chromium Authors.
All rights reserved.
I would like to move this file into testing/android/native_test/, but I have no
idea how to do so since it's directly used by blink and webrtc.

[Yaron, 2015-04-08 16:29:56]

On 2015/04/08 16:01:02, jbudorick wrote:
>
https://codereview.chromium.org/1071573003/diff/1/testing/android/native_test...
> File testing/android/native_test.gyp (right):
> 
>
https://codereview.chromium.org/1071573003/diff/1/testing/android/native_test...
> testing/android/native_test.gyp:1: # Copyright (c) 2012 The Chromium Authors.
> All rights reserved.
> I would like to move this file into testing/android/native_test/, but I have
no
> idea how to do so since it's directly used by blink and webrtc.

lgtm.

You'd have to do a 3-sided patch. Leave the old target which just depends on the
new target, let that roll, update them, and remove the forwarding target

[Ted C, 2015-04-08 17:24:30]

lgtm

[jbudorick, 2015-04-08 17:27:43]

jbudorick@chromium.org changed reviewers:
+ dpranke@chromium.org

[jbudorick, 2015-04-08 17:27:44]

+dpranke for remaining OWNERS

[Dirk Pranke, 2015-04-08 18:46:05]

lgtm

[jbudorick, 2015-04-08 18:58:44]

The CQ bit was checked by jbudorick@chromium.org

[commit-bot: I haz the power, 2015-04-08 19:00:57]

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

[commit-bot: I haz the power, 2015-04-08 23:15:34]

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

[commit-bot: I haz the power, 2015-04-08 23:15:35]

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

[jbudorick, 2015-04-09 04:35:55]

jbudorick@chromium.org changed reviewers:
+ jamiewalch@chromium.org, jln@chromium.org, sievers@chromium.org,
thakis@chromium.org, xhwang@chromium.org

[jbudorick, 2015-04-09 04:35:56]

Guess I misunderstood the per-file BUILD.gn in src/OWNERS.

Adding OWNERS for BUILD.gn files:
 +thakis for base/ and chrome/test/
 +sievers for content/test/ and gpu/
 +xhwang for media/
 +jamiewalch for remoting/
 +jln for sandbox/linux/

[Nico, 2015-04-09 05:09:00]

base, build, chrome lgtm

Please say in the CL description why this is done. Cl descriptions should always
answer "why?" or link to a bug that answers that question.

[jbudorick, 2015-04-09 13:18:15]

On 2015/04/09 05:09:00, Nico wrote:
> base, build, chrome lgtm
> 
> Please say in the CL description why this is done. Cl descriptions should
always
> answer "why?" or link to a bug that answers that question.

Done.

[xhwang, 2015-04-09 16:34:50]

media/ lgtm

[no sievers, 2015-04-09 17:55:13]

On 2015/04/09 04:35:56, jbudorick wrote:
>  +sievers for content/test/ and gpu/

lgtm

[jln (very slow on Chromium), 2015-04-09 18:47:54]

sandbox/ lgtm

[Lambros, 2015-04-09 19:12:02]

lambroslambrou@chromium.org changed reviewers:
+ lambroslambrou@chromium.org

[Lambros, 2015-04-09 19:12:02]

remoting/ lgtm

[jbudorick, 2015-04-09 19:39:30]

The CQ bit was checked by jbudorick@chromium.org

[commit-bot: I haz the power, 2015-04-09 19:40:38]

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

[commit-bot: I haz the power, 2015-04-09 21:52:10]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-04-09 21:53:16]

Patchset 3 (id:??) landed as
https://crrev.com/6ab22495db53ab1a423e90f9f698c36a92f9eb62
Cr-Commit-Position: refs/heads/master@{#324512}