Isolate ui_touch_selection_unittests.

Isolate ui_touch_selection_unittests.

BUG=525873
Committed: https://crrev.com/4c3497591aba5d15f8f0caee9aca2b98d53ed955
Cr-Commit-Position: refs/heads/master@{#353873}

[ghost stip (do not use), 2015-09-29 18:07:08]

stip@chromium.org changed reviewers:
+ jbudorick@chromium.org, jdduke@chromium.org, maruel@chromium.org

[ghost stip (do not use), 2015-09-29 18:11:20]

maruel/jbudorick: review
jdduke: OWNERS

[jbudorick, 2015-09-29 18:14:38]

https://codereview.chromium.org/1374843004/diff/1/ui/touch_selection/ui_touch...
File ui/touch_selection/ui_touch_selection.gyp (right):

https://codereview.chromium.org/1374843004/diff/1/ui/touch_selection/ui_touch...
ui/touch_selection/ui_touch_selection.gyp:135: 'target_name':
'ui_touch_selection_unittests_run',
Do we want this to build on android?

[jdduke (slow), 2015-09-29 18:25:48]

What about GN?

https://codereview.chromium.org/1374843004/diff/1/ui/touch_selection/ui_touch...
File ui/touch_selection/ui_touch_selection.gyp (right):

https://codereview.chromium.org/1374843004/diff/1/ui/touch_selection/ui_touch...
ui/touch_selection/ui_touch_selection.gyp:159: 'target_name':
'ui_touch_selection_unittests_apk_run',
Are we doing this for each test apk separately? I notice we might have the same
issue in events.gyp?

[ghost stip (do not use), 2015-09-29 21:50:09]

https://chromiumcodereview.appspot.com/1374843004/diff/1/ui/touch_selection/u...
File ui/touch_selection/ui_touch_selection.gyp (right):

https://chromiumcodereview.appspot.com/1374843004/diff/1/ui/touch_selection/u...
ui/touch_selection/ui_touch_selection.gyp:135: 'target_name':
'ui_touch_selection_unittests_run',
On 2015/09/29 18:14:38, jbudorick wrote:
> Do we want this to build on android?

Are you saying I should disable this on android? I'm not sure it really matters
here, but I'm happy to do that if you think it's needed

https://chromiumcodereview.appspot.com/1374843004/diff/1/ui/touch_selection/u...
ui/touch_selection/ui_touch_selection.gyp:159: 'target_name':
'ui_touch_selection_unittests_apk_run',
On 2015/09/29 18:25:48, jdduke wrote:
> Are we doing this for each test apk separately? I notice we might have the
same
> issue in events.gyp?

yeah, mainly for OWNERS reasons (and I hate reviewing huge CLs).

[jbudorick, 2015-09-30 08:49:41]

https://codereview.chromium.org/1374843004/diff/1/ui/touch_selection/ui_touch...
File ui/touch_selection/ui_touch_selection.gyp (right):

https://codereview.chromium.org/1374843004/diff/1/ui/touch_selection/ui_touch...
ui/touch_selection/ui_touch_selection.gyp:135: 'target_name':
'ui_touch_selection_unittests_run',
On 2015/09/29 at 21:50:09, stip wrote:
> On 2015/09/29 18:14:38, jbudorick wrote:
> > Do we want this to build on android?
> 
> Are you saying I should disable this on android? I'm not sure it really
matters here, but I'm happy to do that if you think it's needed

Yes.

[ghost stip (do not use), 2015-10-13 06:39:20]

ptal

https://codereview.chromium.org/1374843004/diff/1/ui/touch_selection/ui_touch...
File ui/touch_selection/ui_touch_selection.gyp (right):

https://codereview.chromium.org/1374843004/diff/1/ui/touch_selection/ui_touch...
ui/touch_selection/ui_touch_selection.gyp:135: 'target_name':
'ui_touch_selection_unittests_run',
On 2015/09/30 08:49:41, jbudorick wrote:
> On 2015/09/29 at 21:50:09, stip wrote:
> > On 2015/09/29 18:14:38, jbudorick wrote:
> > > Do we want this to build on android?
> > 
> > Are you saying I should disable this on android? I'm not sure it really
> matters here, but I'm happy to do that if you think it's needed
> 
> Yes.

Done.

[jbudorick, 2015-10-13 13:09:46]

lgtm

[jdduke (slow), 2015-10-13 21:05:02]

rubberstamp lgtm

[ghost stip (do not use), 2015-10-13 21:07:00]

The CQ bit was checked by stip@chromium.org

[commit-bot: I haz the power, 2015-10-13 21:11:33]

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

[commit-bot: I haz the power, 2015-10-13 22:23:35]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-13 22:24:30]

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