Make Omnibox stay visible during long page loads.

Make Omnibox stay visible during long page loads.

The omnibox visibility timer should not be active during a page load; it
should be started only after loading finishes.

BUG=641508
TEST=Viewed a page that takes 10s to load; omnibox stays visible.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/a38ed6430b5688221a944d9994cc2231405a7360
Cr-Commit-Position: refs/heads/master@{#435007}

[cjgrant, 2016-11-28 16:53:58]

Description was changed from

==========
Make Omnibox stay visible during long page loads.

The omnibox visibility timer should not be active during a page load; it
should be started only after loading finishes.

BUG=641508
TEST=Viewed a page that takes 10s to load; omnibox stays visible.
==========

to

==========
Make Omnibox stay visible during long page loads.

The omnibox visibility timer should not be active during a page load; it
should be started only after loading finishes.

BUG=641508
TEST=Viewed a page that takes 10s to load; omnibox stays visible.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[cjgrant, 2016-11-28 16:55:51]

cjgrant@chromium.org changed reviewers:
+ bshe@chromium.org, mthiesse@chromium.org

[bshe, 2016-11-28 18:01:08]

On 2016/11/28 16:55:51, cjgrant wrote:

lgtm

[mthiesse, 2016-11-28 18:40:48]

lgtm

[cjgrant, 2016-11-28 18:52:51]

The CQ bit was checked by cjgrant@chromium.org

[commit-bot: I haz the power, 2016-11-28 18:53:28]

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-28 19:34:24]

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

[commit-bot: I haz the power, 2016-11-28 19:34:25]

Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[cjgrant, 2016-11-29 14:41:21]

The CQ bit was checked by cjgrant@chromium.org

[commit-bot: I haz the power, 2016-11-29 14:41:46]

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-29 16:18:23]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480430481694900, "parent_rev":
"c507b0dd1715d66bab270db7f436cb9bb3106054", "commit_rev":
"ec6913f44dd984f3f3a6ab081076ec037d8776ee"}

[commit-bot: I haz the power, 2016-11-29 16:18:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-29 16:20:29]

Description was changed from

==========
Make Omnibox stay visible during long page loads.

The omnibox visibility timer should not be active during a page load; it
should be started only after loading finishes.

BUG=641508
TEST=Viewed a page that takes 10s to load; omnibox stays visible.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Make Omnibox stay visible during long page loads.

The omnibox visibility timer should not be active during a page load; it
should be started only after loading finishes.

BUG=641508
TEST=Viewed a page that takes 10s to load; omnibox stays visible.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[commit-bot: I haz the power, 2016-11-29 16:20:30]

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