SELECT element: Commit a provisional OPTION selection on blur.

SELECT element: Commit a provisional OPTION selection on blur.

PopupMenuImpl::hide(), which is called on blur, should have same behavior as
PopupMenuImpl::closePopup(), which is called on canceling a popup.

Also, this CL removes hidePopup() call on 'disabled' attribute change because
enabling 'disabled' attribute will remove the focus, and move blur event handling
from menuListDefaultEventHandler() to dispatchBlurEvent() because
menuListDefaultEventHandler() isn't called if the element is disabled.

BUG=659585
Committed: https://crrev.com/b9ec42c0abbf77c9228f0d97ce261fb16b356094
Cr-Commit-Position: refs/heads/master@{#434395}

[tkent, 2016-11-24 05:35:44]

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

[commit-bot: I haz the power, 2016-11-24 05:36:09]

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

[tkent, 2016-11-24 05:46:36]

Description was changed from

==========
SELECT element: Commit a provisional OPTION selectin on blur.

BUG=
==========

to

==========
SELECT element: Commit a provisional OPTION selection on blur.

PopupMenuImpl::hide(), which is called on blur, should have same behavior as
PopupMenuImpl::closePopup(), which is called on canceling a popup.

Also, this CL removes hidePopup() call on 'disabled' attribute change because
enabling 'disabled' attribute will remove the focus.

BUG=659585
==========

[commit-bot: I haz the power, 2016-11-24 06:39:07]

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

[commit-bot: I haz the power, 2016-11-24 06:39:08]

Dry run: Try jobs failed on following builders:
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[tkent, 2016-11-24 07:21:31]

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

[commit-bot: I haz the power, 2016-11-24 07:21:56]

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

[tkent, 2016-11-24 07:21:57]

Patchset #1 (id:1) has been deleted

[tkent, 2016-11-24 07:23:56]

Description was changed from

==========
SELECT element: Commit a provisional OPTION selection on blur.

PopupMenuImpl::hide(), which is called on blur, should have same behavior as
PopupMenuImpl::closePopup(), which is called on canceling a popup.

Also, this CL removes hidePopup() call on 'disabled' attribute change because
enabling 'disabled' attribute will remove the focus.

BUG=659585
==========

to

==========
SELECT element: Commit a provisional OPTION selection on blur.

PopupMenuImpl::hide(), which is called on blur, should have same behavior as
PopupMenuImpl::closePopup(), which is called on canceling a popup.

Also, this CL removes hidePopup() call on 'disabled' attribute change because
enabling 'disabled' attribute will remove the focus, and move blur event
handling
from menuListDefaultEventHandler() to dispatchBlurEvent() because
menuListDefaultEventHandler() isn't called if the element is disabled.

BUG=659585
==========

[tkent, 2016-11-24 07:24:43]

tkent@chromium.org changed reviewers:
+ keishi@chromium.org

[tkent, 2016-11-24 07:24:44]

Keishi, would you review this please?

[keishi, 2016-11-24 08:02:02]

LGTM

[commit-bot: I haz the power, 2016-11-24 09:22:56]

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

[commit-bot: I haz the power, 2016-11-24 09:22:57]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
  chromeos_x86-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT, no
build URL)

[tkent, 2016-11-24 13:21:59]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-11-24 13:22:25]

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-11-24 15:16:39]

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

[commit-bot: I haz the power, 2016-11-24 15:16:40]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[tkent, 2016-11-24 15:44:12]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-11-24 15:44:45]

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-11-24 16:34:45]

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

[commit-bot: I haz the power, 2016-11-24 16:34:46]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[tkent, 2016-11-24 20:51:43]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-11-24 20:52:01]

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-11-24 21:51:36]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1480020703798340,
"parent_rev": "90b6bc00c2f4f6a2ec4f679f6e2ce2eb9c675ebd", "commit_rev":
"bc1c1f04833798b44c23a37b91e8a77e2e8b3b26"}

[commit-bot: I haz the power, 2016-11-24 21:52:00]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-11-24 21:54:02]

Description was changed from

==========
SELECT element: Commit a provisional OPTION selection on blur.

PopupMenuImpl::hide(), which is called on blur, should have same behavior as
PopupMenuImpl::closePopup(), which is called on canceling a popup.

Also, this CL removes hidePopup() call on 'disabled' attribute change because
enabling 'disabled' attribute will remove the focus, and move blur event
handling
from menuListDefaultEventHandler() to dispatchBlurEvent() because
menuListDefaultEventHandler() isn't called if the element is disabled.

BUG=659585
==========

to

==========
SELECT element: Commit a provisional OPTION selection on blur.

PopupMenuImpl::hide(), which is called on blur, should have same behavior as
PopupMenuImpl::closePopup(), which is called on canceling a popup.

Also, this CL removes hidePopup() call on 'disabled' attribute change because
enabling 'disabled' attribute will remove the focus, and move blur event
handling
from menuListDefaultEventHandler() to dispatchBlurEvent() because
menuListDefaultEventHandler() isn't called if the element is disabled.

BUG=659585

Committed: https://crrev.com/b9ec42c0abbf77c9228f0d97ce261fb16b356094
Cr-Commit-Position: refs/heads/master@{#434395}
==========

[commit-bot: I haz the power, 2016-11-24 21:54:03]

Patchset 1 (id:??) landed as
https://crrev.com/b9ec42c0abbf77c9228f0d97ce261fb16b356094
Cr-Commit-Position: refs/heads/master@{#434395}