[Home] Hide sheet when back pressed

[Home] Hide sheet when back pressed

This change allows a single press of the back button to close the
bottom sheet if expanded via omnibox focus. This allows a user to
easily return to the page if they decide against using the omnibox.

The mechanism for detecting this case is simple: if the placeholder
content is showing, defocus and hide everything with one press. The
reason this works is because the placeholder sheet content will only
be showing if the user expanded the sheet from the peeking state by
focusing the omnibox.

BUG=716224
Review-Url: https://codereview.chromium.org/2896483003
Cr-Commit-Position: refs/heads/master@{#473212}
Committed: https://chromium.googlesource.com/chromium/src/+/e309521b316083148b2b2073fb589a086e010bc7

[mdjones, 2017-05-18 20:46:26]

mdjones@chromium.org changed reviewers:
+ twellington@chromium.org

[mdjones, 2017-05-18 20:46:27]

ptal

[Theresa, 2017-05-18 20:48:40]

lgtm

[mdjones, 2017-05-18 20:49:15]

mdjones@chromium.org changed reviewers:
+ tedchoc@chromium.org

[mdjones, 2017-05-18 20:49:16]

+tedchoc ptal

[Ted C, 2017-05-18 23:48:45]

lgtm

[mdjones, 2017-05-19 16:03:12]

The CQ bit was checked by mdjones@chromium.org

[commit-bot: I haz the power, 2017-05-19 16:03:39]

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-05-19 16:45:24]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495209792655020,
"parent_rev": "fd8d9f7d5fade562a24f30ceadaa1d2b540d810e", "commit_rev":
"e309521b316083148b2b2073fb589a086e010bc7"}

[commit-bot: I haz the power, 2017-05-19 16:45:35]

Description was changed from

==========
[Home] Hide sheet when back pressed

This change allows a single press of the back button to close the
bottom sheet if expanded via omnibox focus. This allows a user to
easily return to the page if they decide against using the omnibox.

The mechanism for detecting this case is simple: if the placeholder
content is showing, defocus and hide everything with one press. The
reason this works is because the placeholder sheet content will only
be showing if the user expanded the sheet from the peeking state by
focusing the omnibox.

BUG=716224
==========

to

==========
[Home] Hide sheet when back pressed

This change allows a single press of the back button to close the
bottom sheet if expanded via omnibox focus. This allows a user to
easily return to the page if they decide against using the omnibox.

The mechanism for detecting this case is simple: if the placeholder
content is showing, defocus and hide everything with one press. The
reason this works is because the placeholder sheet content will only
be showing if the user expanded the sheet from the peeking state by
focusing the omnibox.

BUG=716224

Review-Url: https://codereview.chromium.org/2896483003
Cr-Commit-Position: refs/heads/master@{#473212}
Committed:
https://chromium.googlesource.com/chromium/src/+/e309521b316083148b2b2073fb58...
==========

[commit-bot: I haz the power, 2017-05-19 16:45:36]

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