Fixed the issue shows keyword hint when omnibox loses its focus.

Fixed the issue that shows keyword hint when omnibox loses its focus.

Keyword hint displays a message 'tab for search', but it cannot use
when omnibox loses its focus. Checking focus states for determining
the visibility of keyword hint in Layout(), and calling layout() when
focus changes would solve this issue.

This patch is written for OSX Chrome, patches for other platforms
may be around here. :)

BUG=649938

[cwdoh, 2016-10-01 03:53:35]

Description was changed from

==========
Fixed the issue that show keyword hint when omnibox loses its focus.

Keyword hint display a message 'tab for search', but it cannot use
when omnibox loses its focus. Checking focus states for determining
the visibility of keyword hint in Layout(), and calling layout() when
focus changes would solve this issue.

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

to

==========
Fixed the issue that shows keyword hint when omnibox loses its focus.

Keyword hint display a message 'tab for search', but it cannot use
when omnibox loses its focus. Checking focus states for determining
the visibility of keyword hint in Layout(), and calling layout() when
focus changes would solve this issue.

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

[cwdoh, 2016-10-01 03:53:35]

changwook.doh@gmail.com changed reviewers:
+ simonhong@chromium.org

[cwdoh, 2016-10-01 03:55:25]

PTAL

[cwdoh, 2016-10-04 02:29:55]

Description was changed from

==========
Fixed the issue that shows keyword hint when omnibox loses its focus.

Keyword hint display a message 'tab for search', but it cannot use
when omnibox loses its focus. Checking focus states for determining
the visibility of keyword hint in Layout(), and calling layout() when
focus changes would solve this issue.

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

to

==========
Fixed the issue that shows keyword hint when omnibox loses its focus.

Keyword hint display a message 'tab for search', but it cannot use
when omnibox loses its focus. Checking focus states for determining
the visibility of keyword hint in Layout(), and calling layout() when
focus changes would solve this issue.

BUG=649938
==========

[cwdoh, 2016-10-04 02:29:56]

changwook.doh@gmail.com changed reviewers:
+ pkasting@google.com
- simonhong@chromium.org

[cwdoh, 2016-10-04 02:30:44]

Description was changed from

==========
Fixed the issue that shows keyword hint when omnibox loses its focus.

Keyword hint display a message 'tab for search', but it cannot use
when omnibox loses its focus. Checking focus states for determining
the visibility of keyword hint in Layout(), and calling layout() when
focus changes would solve this issue.

BUG=649938
==========

to

==========
Fixed the issue that shows keyword hint when omnibox loses its focus.

Keyword hint displays a message 'tab for search', but it cannot use
when omnibox loses its focus. Checking focus states for determining
the visibility of keyword hint in Layout(), and calling layout() when
focus changes would solve this issue.

BUG=649938
==========

[cwdoh, 2016-10-04 02:33:23]

Description was changed from

==========
Fixed the issue that shows keyword hint when omnibox loses its focus.

Keyword hint displays a message 'tab for search', but it cannot use
when omnibox loses its focus. Checking focus states for determining
the visibility of keyword hint in Layout(), and calling layout() when
focus changes would solve this issue.

BUG=649938
==========

to

==========
Fixed the issue that shows keyword hint when omnibox loses its focus.

Keyword hint displays a message 'tab for search', but it cannot use
when omnibox loses its focus. Checking focus states for determining
the visibility of keyword hint in Layout(), and calling layout() when
focus changes would solve this issue.

This patch is written for OSX Chrome, patches for other platforms may be around
here. :)

BUG=649938
==========

[cwdoh, 2016-10-04 02:33:37]

Description was changed from

==========
Fixed the issue that shows keyword hint when omnibox loses its focus.

Keyword hint displays a message 'tab for search', but it cannot use
when omnibox loses its focus. Checking focus states for determining
the visibility of keyword hint in Layout(), and calling layout() when
focus changes would solve this issue.

This patch is written for OSX Chrome, patches for other platforms may be around
here. :)

BUG=649938
==========

to

==========
Fixed the issue that shows keyword hint when omnibox loses its focus.

Keyword hint displays a message 'tab for search', but it cannot use
when omnibox loses its focus. Checking focus states for determining
the visibility of keyword hint in Layout(), and calling layout() when
focus changes would solve this issue.

This patch is written for OSX Chrome, patches for other platforms
may be around here. :)

BUG=649938
==========

[cwdoh, 2016-10-04 02:34:50]

PTAL.

[cwdoh, 2016-10-04 04:16:09]

The CQ bit was checked by changwook.doh@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2016-10-04 04:16:28]

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, 2016-10-04 04:16:29]

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

[commit-bot: I haz the power, 2016-10-04 04:16:29]

Dry run: No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Peter Kasting, 2016-10-05 00:20:36]

pkasting@chromium.org changed reviewers:
+ pkasting@chromium.org
- pkasting@google.com

[Peter Kasting, 2016-10-05 00:22:17]

Are you aware of https://codereview.chromium.org/2373773002/ , which fixes this
on the views side only?

Please see the comments there and work with the author of that patch, ideally to
produce one combined patch which addresses all platforms (but two patches are OK
if they're designed with each other in mind).

[Peter Kasting, 2017-02-11 01:58:03]

What's the status of this CL?

[Peter Kasting, 2017-08-29 03:38:20]

Closing; no response from author, other patch landed, bug is marked fixed