[Merge to 2924] Reland "Propagate information about how ARC apps are launched"

[Merge to 2924] Reland "Propagate information about how ARC apps are launched"

This change lets ARC know whether an app should be launched in Touch
Mode[1] or not.

1: http://android-developers.blogspot.com/2008/12/touch-mode.html

BUG=669146
TEST=ARC app starts in touch mode when activated using
     mouse/trackpad/touch
TEST=ARC app starts in focus mode when activated using
     keyboard.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/b5898c35b2415dbf8b5d9aaa19a6b13b0510d1e7
Review-Url: https://codereview.chromium.org/2540433002
Cr-Original-Commit-Position: refs/heads/master@{#435115}
Cr-Commit-Position: refs/heads/master@{#435327}
(cherry picked from commit fa0327a360d5e676680a4b5337a607958c0512d0)

R=stevenjb@chromium.org

Committed: https://chromium.googlesource.com/chromium/src/+/d524b674124fb04275466fb0060585d8b0b427eb

[Luis Héctor Chávez, 2016-12-02 13:44:29]

lhchavez@chromium.org changed reviewers:
+ stevenjb@chromium.org

[Luis Héctor Chávez, 2016-12-02 13:44:30]

This is a merge to M56, but didn't merge cleanly (some conflicts with a
refactoring to split ArcAuthService). PTAL

[stevenjb, 2016-12-02 16:52:51]

RS LGTM (unless there is a specific area you would like me to double check).
Typically straightforward merges can use TBR=original-reviewer.

[stevenjb, 2016-12-02 16:56:12]

On 2016/12/02 16:52:51, stevenjb wrote:
> RS LGTM (unless there is a specific area you would like me to double check).
> Typically straightforward merges can use TBR=original-reviewer.

BTW, is there an expected difference in how the Android settings app behaves
when launched with ui::EF_LEFT_MOUSE_BUTTON vs ui::EF_NONE? I added you to
https://codereview.chromium.org/2541923002/ to look at the implementation for MD
Settings.

[Luis Héctor Chávez, 2016-12-02 18:44:31]

The CQ bit was checked by lhchavez@chromium.org

[commit-bot: I haz the power, 2016-12-02 18:44:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-02 18:44:57]

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

[commit-bot: I haz the power, 2016-12-02 18:44:58]

CLs for remote refs other than refs/pending/heads/master must contain NOTRY=true
and NOPRESUBMIT=true in order for the CQ to process them

[Luis Héctor Chávez, 2016-12-02 18:48:00]

On 2016/12/02 16:56:12, stevenjb wrote:
> On 2016/12/02 16:52:51, stevenjb wrote:
> > RS LGTM (unless there is a specific area you would like me to double check).
> > Typically straightforward merges can use TBR=original-reviewer.
> 
> BTW, is there an expected difference in how the Android settings app behaves
> when launched with ui::EF_LEFT_MOUSE_BUTTON vs ui::EF_NONE? I added you to
> https://codereview.chromium.org/2541923002/ to look at the implementation for
MD
> Settings.

Yes, when launched with ui::EF_LEFT_MOUSE_BUTTON it should enter touch mode. For
the settings app the only noticeable differecen AFAICT is that when it's out of
touch mode (when launched with ui::EF_NONE) it should show focus around one
element, to hint that it's navigable with the keyboard.

[Luis Héctor Chávez, 2016-12-02 18:57:37]

Description was changed from

==========
[Merge to 2924] Reland "Propagate information about how ARC apps are launched"

This change lets ARC know whether an app should be launched in Touch
Mode[1] or not.

1: http://android-developers.blogspot.com/2008/12/touch-mode.html

BUG=669146
TEST=ARC app starts in touch mode when activated using
     mouse/trackpad/touch
TEST=ARC app starts in focus mode when activated using
     keyboard.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/b5898c35b2415dbf8b5d9aaa19a6b13b0510d1e7
Review-Url: https://codereview.chromium.org/2540433002
Cr-Original-Commit-Position: refs/heads/master@{#435115}
Cr-Commit-Position: refs/heads/master@{#435327}
(cherry picked from commit fa0327a360d5e676680a4b5337a607958c0512d0)
==========

to

==========
[Merge to 2924] Reland "Propagate information about how ARC apps are launched"

This change lets ARC know whether an app should be launched in Touch
Mode[1] or not.

1: http://android-developers.blogspot.com/2008/12/touch-mode.html

BUG=669146
TEST=ARC app starts in touch mode when activated using
     mouse/trackpad/touch
TEST=ARC app starts in focus mode when activated using
     keyboard.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/b5898c35b2415dbf8b5d9aaa19a6b13b0510d1e7
Review-Url: https://codereview.chromium.org/2540433002
Cr-Original-Commit-Position: refs/heads/master@{#435115}
Cr-Commit-Position: refs/heads/master@{#435327}
(cherry picked from commit fa0327a360d5e676680a4b5337a607958c0512d0)

R=stevenjb@chromium.org

Committed:
https://chromium.googlesource.com/chromium/src/+/d524b674124fb04275466fb00605...
==========

[Luis Héctor Chávez, 2016-12-02 18:57:38]

Committed patchset #1 (id:1) manually as
d524b674124fb04275466fb0060585d8b0b427eb (presubmit successful).