[Android Omnibox] Fixing Emphasizer missing ConnectionSecurityLevel entry in switch

[Android Omnibox] Fixing Emphasizer missing ConnectionSecurityLevel entry in switch

Adding a case statement for HTTP_SHOW_WARNING with a fall-through same
as for NONE per usage in LocationBarLayout.java

BUG=681115
R=tedchoc@chromium.org

Review-Url: https://codereview.chromium.org/2629543011
Cr-Commit-Position: refs/heads/master@{#443733}
Committed: https://chromium.googlesource.com/chromium/src/+/91942aefbedd1909d0ab28a87bda8603ca99b87c

[fgorski, 2017-01-13 19:25:19]

Ted, can you take a look and let me know if we need security team to look at it?
(Adding Emily to CC just in case we do)

[fgorski, 2017-01-13 19:38:00]

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

[commit-bot: I haz the power, 2017-01-13 19:38:57]

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-01-13 20:30:07]

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

[commit-bot: I haz the power, 2017-01-13 20:30:09]

Dry run: This issue passed the CQ dry run.

[fgorski, 2017-01-13 21:36:35]

Description was changed from

==========
[Android Omnibox] Fixing Emphasizer missing ConnectionSecurityLevel entry in
switch

Adding a case statement for HTTP_SHOW_WARNING with a fall-through same
as for NONE per usage in LocationBarLayout.java

BUG=681115
R=tedchoc@chromium.org
==========

to

==========
[Android Omnibox] Fixing Emphasizer missing ConnectionSecurityLevel entry in
switch

Adding a case statement for HTTP_SHOW_WARNING with a fall-through same
as for NONE per usage in LocationBarLayout.java

BUG=681115
R=tedchoc@chromium.org
==========

[Ted C, 2017-01-13 23:39:16]

On 2017/01/13 19:25:19, fgorski wrote:
> Ted, can you take a look and let me know if we need security team to look at
it?
> (Adding Emily to CC just in case we do)

lgtm, I would think this is fine to land.

[fgorski, 2017-01-13 23:55:12]

The CQ bit was checked by fgorski@chromium.org

[commit-bot: I haz the power, 2017-01-13 23:55:48]

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-01-14 00:01:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484351712747650, "parent_rev":
"d7658fd0442cce20cb16ada469d812a94655270a", "commit_rev":
"91942aefbedd1909d0ab28a87bda8603ca99b87c"}

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

Description was changed from

==========
[Android Omnibox] Fixing Emphasizer missing ConnectionSecurityLevel entry in
switch

Adding a case statement for HTTP_SHOW_WARNING with a fall-through same
as for NONE per usage in LocationBarLayout.java

BUG=681115
R=tedchoc@chromium.org
==========

to

==========
[Android Omnibox] Fixing Emphasizer missing ConnectionSecurityLevel entry in
switch

Adding a case statement for HTTP_SHOW_WARNING with a fall-through same
as for NONE per usage in LocationBarLayout.java

BUG=681115
R=tedchoc@chromium.org

Review-Url: https://codereview.chromium.org/2629543011
Cr-Commit-Position: refs/heads/master@{#443733}
Committed:
https://chromium.googlesource.com/chromium/src/+/91942aefbedd1909d0ab28a87bda...
==========

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

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/91942aefbedd1909d0ab28a87bda...