MD Settings: Focus the container, instead of settings-main, when sidenav is closed.

MD Settings: Focus the container, instead of settings-main, when sidenav is closed.

This fixes a regression where the scroll position changed after closing the
sidenav, and is more correct, since the container is the scroll target.

BUG=708960
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2802073002
Cr-Commit-Position: refs/heads/master@{#462913}
Committed: https://chromium.googlesource.com/chromium/src/+/c3257e6eb48627fde7ad129012ee9904110e4969

[dpapad, 2017-04-06 18:46:23]

Description was changed from

==========
MD Settings: Focus the container, instead of settings-main, when sidenav is
closed.

This is more correct, since the container is what is actually being scrolled.

BUG=708960
==========

to

==========
MD Settings: Focus the container, instead of settings-main, when sidenav is
closed.

This is more correct, since the container is what is actually being scrolled.

BUG=708960
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[dpapad, 2017-04-06 18:47:10]

Description was changed from

==========
MD Settings: Focus the container, instead of settings-main, when sidenav is
closed.

This is more correct, since the container is what is actually being scrolled.

BUG=708960
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: Focus the container, instead of settings-main, when sidenav is
closed.

This fixes a regression where the scroll position changed after closing the
sidenav, and is more correct, since the container is what is the scroll target.

BUG=708960
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[dpapad, 2017-04-06 18:47:18]

Description was changed from

==========
MD Settings: Focus the container, instead of settings-main, when sidenav is
closed.

This fixes a regression where the scroll position changed after closing the
sidenav, and is more correct, since the container is what is the scroll target.

BUG=708960
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: Focus the container, instead of settings-main, when sidenav is
closed.

This fixes a regression where the scroll position changed after closing the
sidenav, and is more correct, since the container is what is the scroll target.

BUG=708960
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[dpapad, 2017-04-06 18:48:10]

dpapad@chromium.org changed reviewers:
+ dbeam@chromium.org

[dpapad, 2017-04-06 18:48:22]

The CQ bit was checked by dpapad@chromium.org to run a CQ dry run

[dpapad, 2017-04-06 18:48:52]

Description was changed from

==========
MD Settings: Focus the container, instead of settings-main, when sidenav is
closed.

This fixes a regression where the scroll position changed after closing the
sidenav, and is more correct, since the container is what is the scroll target.

BUG=708960
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: Focus the container, instead of settings-main, when sidenav is
closed.

This fixes a regression where the scroll position changed after closing the
sidenav, and is more correct, since the container is the scroll target.

BUG=708960
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2017-04-06 18:49:41]

Dry run: 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-04-06 19:58:56]

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

[commit-bot: I haz the power, 2017-04-06 19:58:57]

Dry run: This issue passed the CQ dry run.

[dpapad, 2017-04-07 16:51:13]

On 2017/04/06 at 18:48:10, dpapad wrote:
> 

Friendly ping. Based on our discussion yesterday, let's land this to fix the
regression, and we can deal with the keyboard use-case in a follow up CL.

[Dan Beam, 2017-04-07 17:22:51]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2017-04-07 17:22:52]

lgtm

[commit-bot: I haz the power, 2017-04-07 17:23:22]

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-04-07 17:40:03]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491585771831020, "parent_rev":
"d5acadb2d99dfbbde0cbe251173b566ceb5d3441", "commit_rev":
"c3257e6eb48627fde7ad129012ee9904110e4969"}

[commit-bot: I haz the power, 2017-04-07 17:40:53]

Description was changed from

==========
MD Settings: Focus the container, instead of settings-main, when sidenav is
closed.

This fixes a regression where the scroll position changed after closing the
sidenav, and is more correct, since the container is the scroll target.

BUG=708960
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: Focus the container, instead of settings-main, when sidenav is
closed.

This fixes a regression where the scroll position changed after closing the
sidenav, and is more correct, since the container is the scroll target.

BUG=708960
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[commit-bot: I haz the power, 2017-04-07 17:40:54]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/c3257e6eb48627fde7ad129012ee...