Renames mandoline_frame_apptests to web_view_apptests

Renames mandoline_frame_apptests to web_view_apptests

BUG=none
TEST=none
R=msw@chromium.org

Committed: https://crrev.com/84fec63dbc6667f4c1e03eed4b6f11b4c9865a40
Cr-Commit-Position: refs/heads/master@{#347984}

[msw, 2015-09-09 18:11:47]

https://codereview.chromium.org/1323333004/diff/1/components/web_view/BUILD.gn
File components/web_view/BUILD.gn (right):

https://codereview.chromium.org/1323333004/diff/1/components/web_view/BUILD.g...
components/web_view/BUILD.gn:86: output_name = "web_view_apptests"
I think we need to specify the name in the target decl, but maybe that was just
android unit tests; I can't remember specifics, let me try to find the CL.

[sky, 2015-09-09 18:14:28]

https://codereview.chromium.org/1323333004/diff/1/components/web_view/BUILD.gn
File components/web_view/BUILD.gn (right):

https://codereview.chromium.org/1323333004/diff/1/components/web_view/BUILD.g...
components/web_view/BUILD.gn:86: output_name = "web_view_apptests"
On 2015/09/09 18:11:47, msw wrote:
> I think we need to specify the name in the target decl, but maybe that was
just
> android unit tests; I can't remember specifics, let me try to find the CL.

We aren't for html_viewer_apptests. But then we don't run html_viewer_apptests
on android yet. And we don't run these tests on android yet either. That said,
if Android still needs the long name than I'll fix both. Let me know.

[msw, 2015-09-09 18:20:22]

lgtm

https://codereview.chromium.org/1323333004/diff/1/components/web_view/BUILD.gn
File components/web_view/BUILD.gn (right):

https://codereview.chromium.org/1323333004/diff/1/components/web_view/BUILD.g...
components/web_view/BUILD.gn:86: output_name = "web_view_apptests"
On 2015/09/09 18:14:28, sky wrote:
> On 2015/09/09 18:11:47, msw wrote:
> > I think we need to specify the name in the target decl, but maybe that was
> just
> > android unit tests; I can't remember specifics, let me try to find the CL.
> 
> We aren't for html_viewer_apptests. But then we don't run html_viewer_apptests
> on android yet. And we don't run these tests on android yet either. That said,
> if Android still needs the long name than I'll fix both. Let me know.

NVM, it's just unit tests for the sake of android *_apk target names:
https://codereview.chromium.org/1195623002

[sky, 2015-09-09 18:20:32]

https://codereview.chromium.org/1323333004/diff/1/components/web_view/BUILD.gn
File components/web_view/BUILD.gn (right):

https://codereview.chromium.org/1323333004/diff/1/components/web_view/BUILD.g...
components/web_view/BUILD.gn:86: output_name = "web_view_apptests"
On 2015/09/09 18:14:28, sky wrote:
> On 2015/09/09 18:11:47, msw wrote:
> > I think we need to specify the name in the target decl, but maybe that was
> just
> > android unit tests; I can't remember specifics, let me try to find the CL.
> 
> We aren't for html_viewer_apptests. But then we don't run html_viewer_apptests
> on android yet. And we don't run these tests on android yet either. That said,
> if Android still needs the long name than I'll fix both. Let me know.

It looks like the view_manager apptests have a target name of 'apptests' and run
on android too. Maybe something changed on the android side to accommodate the
different name?

[msw, 2015-09-09 18:20:47]

oh, one nit...

https://codereview.chromium.org/1323333004/diff/1/mojo/tools/data/apptests
File mojo/tools/data/apptests (right):

https://codereview.chromium.org/1323333004/diff/1/mojo/tools/data/apptests#ne...
mojo/tools/data/apptests:83: 'test': 'mojo:web_view_apptests',
nit: abc order?

[sky, 2015-09-09 18:25:15]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2015-09-09 18:25:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1323333004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1323333004/1

[sky, 2015-09-09 18:33:39]

The CQ bit was unchecked by sky@chromium.org

[sky, 2015-09-09 18:34:37]

https://codereview.chromium.org/1323333004/diff/1/mojo/tools/data/apptests
File mojo/tools/data/apptests (right):

https://codereview.chromium.org/1323333004/diff/1/mojo/tools/data/apptests#ne...
mojo/tools/data/apptests:83: 'test': 'mojo:web_view_apptests',
On 2015/09/09 18:20:46, msw wrote:
> nit: abc order?

Done.

[sky, 2015-09-09 18:34:57]

The CQ bit was checked by sky@chromium.org

[sky, 2015-09-09 18:34:57]

The patchset sent to the CQ was uploaded after l-g-t-m from msw@chromium.org
Link to the patchset: https://codereview.chromium.org/1323333004/#ps20001
(title: "order")

[commit-bot: I haz the power, 2015-09-09 18:35:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1323333004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1323333004/20001

[commit-bot: I haz the power, 2015-09-09 19:43:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-09 19:44:29]

Patchset 2 (id:??) landed as
https://crrev.com/84fec63dbc6667f4c1e03eed4b6f11b4c9865a40
Cr-Commit-Position: refs/heads/master@{#347984}

[commit-bot: I haz the power, 2015-09-23 12:02:17]

Patchset 2 (id:??) landed as
https://crrev.com/84fec63dbc6667f4c1e03eed4b6f11b4c9865a40
Cr-Commit-Position: refs/heads/master@{#347984}