[Merge M58] arc: Fix regression in handling ARC shelf item.

[Merge M58] arc: Fix regression in handling ARC shelf item.

This fix regressions caused by recent refactoring. First regression is
in case multiple shelf items for the same ARC app, clicking on shelf
item always activates last opened window. It is expected that in this
case last active window of that app becomes active. Second one,
selecting which item to activate from menu does not actually activate
selected item

BUG=703895
TEST=Manually, unit_tests extended.
NOTRY=true
NOPRESUBMIT=true
TBR=msw@chromium.org


Review-Url: https://codereview.chromium.org/2763893004
Cr-Commit-Position: refs/heads/master@{#458638}
(cherry picked from commit a19209825989de7d6b620aea95dd0f95b5c3c03f)

Review-Url: https://codereview.chromium.org/2773493002
Cr-Commit-Position: refs/branch-heads/3029@{#377}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}
Committed: https://chromium.googlesource.com/chromium/src/+/e62cb970ec7675219a52e1f529136335fc95970d

[khmel, 2017-03-23 01:13:53]

Description was changed from

==========
[Merge M58] arc: Fix regression in handling ARC shelf item.

This fix regressions caused by recent refactoring. First regression is
in case multiple shelf items for the same ARC app, clicking on shelf
item always activates last opened window. It is expected that in this
case last active window of that app becomes active. Second one,
selecting which item to activate from menu does not actually activate
selected item

BUG=703895
TEST=Manually, unit_tests extended.
TRY=true
NOPRESUBMIT=true
TBR=msw@chromium.org


Review-Url: https://codereview.chromium.org/2763893004
Cr-Commit-Position: refs/heads/master@{#458638}
(cherry picked from commit a19209825989de7d6b620aea95dd0f95b5c3c03f)
==========

to

==========
[Merge M58] arc: Fix regression in handling ARC shelf item.

This fix regressions caused by recent refactoring. First regression is
in case multiple shelf items for the same ARC app, clicking on shelf
item always activates last opened window. It is expected that in this
case last active window of that app becomes active. Second one,
selecting which item to activate from menu does not actually activate
selected item

BUG=703895
TEST=Manually, unit_tests extended.
TRY=true
NOPRESUBMIT=true
TBR=msw@chromium.org


Review-Url: https://codereview.chromium.org/2763893004
Cr-Commit-Position: refs/heads/master@{#458638}
(cherry picked from commit a19209825989de7d6b620aea95dd0f95b5c3c03f)
==========

[khmel, 2017-03-23 01:13:54]

khmel@chromium.org changed reviewers:
- msw@chromium.org

[khmel, 2017-03-23 01:14:10]

passed locally unit_tests + manually tested on device

[khmel, 2017-03-23 01:14:14]

The CQ bit was checked by khmel@chromium.org

[commit-bot: I haz the power, 2017-03-23 01:14:44]

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, 2017-03-23 01:14:45]

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

[commit-bot: I haz the power, 2017-03-23 01:14:45]

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

[khmel, 2017-03-23 01:42:09]

Description was changed from

==========
[Merge M58] arc: Fix regression in handling ARC shelf item.

This fix regressions caused by recent refactoring. First regression is
in case multiple shelf items for the same ARC app, clicking on shelf
item always activates last opened window. It is expected that in this
case last active window of that app becomes active. Second one,
selecting which item to activate from menu does not actually activate
selected item

BUG=703895
TEST=Manually, unit_tests extended.
TRY=true
NOPRESUBMIT=true
TBR=msw@chromium.org


Review-Url: https://codereview.chromium.org/2763893004
Cr-Commit-Position: refs/heads/master@{#458638}
(cherry picked from commit a19209825989de7d6b620aea95dd0f95b5c3c03f)
==========

to

==========
[Merge M58] arc: Fix regression in handling ARC shelf item.

This fix regressions caused by recent refactoring. First regression is
in case multiple shelf items for the same ARC app, clicking on shelf
item always activates last opened window. It is expected that in this
case last active window of that app becomes active. Second one,
selecting which item to activate from menu does not actually activate
selected item

BUG=703895
TEST=Manually, unit_tests extended.
NOTRY=true
NOPRESUBMIT=true
TBR=msw@chromium.org


Review-Url: https://codereview.chromium.org/2763893004
Cr-Commit-Position: refs/heads/master@{#458638}
(cherry picked from commit a19209825989de7d6b620aea95dd0f95b5c3c03f)
==========

[khmel, 2017-03-23 01:42:13]

The CQ bit was checked by khmel@chromium.org

[commit-bot: I haz the power, 2017-03-23 01:42:57]

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, 2017-03-23 01:45:14]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490233333042490,
"parent_rev": "8adb3a48e309f33206b808293a7b17be0bfe5261", "commit_rev":
"e62cb970ec7675219a52e1f529136335fc95970d"}

[commit-bot: I haz the power, 2017-03-23 01:45:57]

Description was changed from

==========
[Merge M58] arc: Fix regression in handling ARC shelf item.

This fix regressions caused by recent refactoring. First regression is
in case multiple shelf items for the same ARC app, clicking on shelf
item always activates last opened window. It is expected that in this
case last active window of that app becomes active. Second one,
selecting which item to activate from menu does not actually activate
selected item

BUG=703895
TEST=Manually, unit_tests extended.
NOTRY=true
NOPRESUBMIT=true
TBR=msw@chromium.org


Review-Url: https://codereview.chromium.org/2763893004
Cr-Commit-Position: refs/heads/master@{#458638}
(cherry picked from commit a19209825989de7d6b620aea95dd0f95b5c3c03f)
==========

to

==========
[Merge M58] arc: Fix regression in handling ARC shelf item.

This fix regressions caused by recent refactoring. First regression is
in case multiple shelf items for the same ARC app, clicking on shelf
item always activates last opened window. It is expected that in this
case last active window of that app becomes active. Second one,
selecting which item to activate from menu does not actually activate
selected item

BUG=703895
TEST=Manually, unit_tests extended.
NOTRY=true
NOPRESUBMIT=true
TBR=msw@chromium.org


Review-Url: https://codereview.chromium.org/2763893004
Cr-Commit-Position: refs/heads/master@{#458638}
(cherry picked from commit a19209825989de7d6b620aea95dd0f95b5c3c03f)

Review-Url: https://codereview.chromium.org/2773493002
Cr-Commit-Position: refs/branch-heads/3029@{#377}
Cr-Branched-From:
939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}
Committed:
https://chromium.googlesource.com/chromium/src/+/e62cb970ec7675219a52e1f52913...
==========

[commit-bot: I haz the power, 2017-03-23 01:45:58]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/e62cb970ec7675219a52e1f52913...