Update the Bling omnibox to be aware of HTTP_SHOW_WARNING and use ShouldAlwaysShowIcon().

Update the Bling omnibox to be aware of HTTP_SHOW_WARNING and use ShouldAlwaysShowIcon().

BUG=647559, 673530
Review-Url: https://codereview.chromium.org/2692173002
Cr-Commit-Position: refs/heads/master@{#454399}
Committed: https://chromium.googlesource.com/chromium/src/+/76c172851a33d32e6eee07701d9ed8821b0e2a30

[lgarron, 2017-02-13 23:51:55]

lgarron@chromium.org changed reviewers:
+ rohitrao@chromium.org

[lgarron, 2017-02-13 23:51:56]

rohitrao@, could you (re-)review?

This is the upstream version of https://chromereviews.googleplex.com/556027013,
unchanged in content but more woke to the environment of today.

[lgarron, 2017-02-14 00:06:33]

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

[commit-bot: I haz the power, 2017-02-14 00:07:10]

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-02-14 00:14:50]

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

[commit-bot: I haz the power, 2017-02-14 00:14:50]

Dry run: Try jobs failed on following builders:
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)

[lgarron, 2017-02-14 22:37:10]

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

[commit-bot: I haz the power, 2017-02-14 22:38:19]

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-02-15 00:40:26]

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

[commit-bot: I haz the power, 2017-02-15 00:40:27]

Dry run: This issue passed the CQ dry run.

[lgarron, 2017-02-27 20:13:35]

On 2017/02/13 at 23:51:56, lgarron wrote:
> rohitrao@, could you (re-)review?
> 
> This is the upstream version of
https://chromereviews.googleplex.com/556027013, unchanged in content but more
woke to the environment of today.

ping :•)

[rohitrao (ping after 24h), 2017-03-02 18:30:09]

lgtm

[lgarron, 2017-03-02 21:04:38]

The CQ bit was checked by lgarron@chromium.org

[commit-bot: I haz the power, 2017-03-02 21:05:15]

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-03-02 22:10:57]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1488488678046470,
"parent_rev": "d5080b74e8ca39517536fc92b3ff78cfbe2a0b3e", "commit_rev":
"76c172851a33d32e6eee07701d9ed8821b0e2a30"}

[commit-bot: I haz the power, 2017-03-02 22:11:35]

Description was changed from

==========
Update the Bling omnibox to be aware of HTTP_SHOW_WARNING and use
ShouldAlwaysShowIcon().

BUG=647559, 673530
==========

to

==========
Update the Bling omnibox to be aware of HTTP_SHOW_WARNING and use
ShouldAlwaysShowIcon().

BUG=647559, 673530

Review-Url: https://codereview.chromium.org/2692173002
Cr-Commit-Position: refs/heads/master@{#454399}
Committed:
https://chromium.googlesource.com/chromium/src/+/76c172851a33d32e6eee07701d9e...
==========

[commit-bot: I haz the power, 2017-03-02 22:11:36]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/76c172851a33d32e6eee07701d9e...