diff options
| author | dbeam@chromium.org <dbeam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-17 12:57:04 +0000 |
|---|---|---|
| committer | dbeam@chromium.org <dbeam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-17 12:57:04 +0000 |
| commit | 127c6a1a41fd69f173bcbcfec5feb7d6cf2c7009 (patch) | |
| tree | 1b3f1665876f4347c9df42c1f1406029c1795cab /skia/ext/platform_device_win.cc | |
| parent | 77a8ca1352fbfb33367064a6a3065444bf21eaf9 (diff) | |
| download | chromium_src-127c6a1a41fd69f173bcbcfec5feb7d6cf2c7009.zip chromium_src-127c6a1a41fd69f173bcbcfec5feb7d6cf2c7009.tar.gz chromium_src-127c6a1a41fd69f173bcbcfec5feb7d6cf2c7009.tar.bz2 | |
Remove invisible views from tab order.
AutofillDialogViews::Layout() and AutofillDialogViews::GetPreferredSize()
short-circuit logic when either the loading shield or sign-in webview are
showing. Unfortunately this has the side-effect of keeping any focusable input
in the dialog in the tab order. Recently I made suggested buttons (down arrows)
focusable so this became more noticeable, but the problem has always been there
(if you currently change all sections to add mode and attempt to sign in you'll
have tons of invisible elements to focus).
This CL makes visually invisible nodes also !visible() to remove them from the
tab order (see: views::View::IsFocusable()).
R=isherman@chromium.org
BUG=306484
Review URL: https://codereview.chromium.org/27085002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@229111 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'skia/ext/platform_device_win.cc')
0 files changed, 0 insertions, 0 deletions