Switch warning_explanations (yellow) to be unauthenticated_explanations (neutral).

Switch warning_explanations (yellow) to be unauthenticated_explanations (neutral).

BUG=534681
Committed: https://crrev.com/8555a6c0f5a20b3409de73b7b120d5701b4668b2
Cr-Commit-Position: refs/heads/master@{#360489}

[lgarron, 2015-11-06 21:59:26]

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

[lgarron, 2015-11-06 21:59:27]

estark@: I've had this hanging around for a while.

The per-request data has been sending "neutral" instead of "warning", but we
never got around to it for this.

Does this look good to you?

[estark, 2015-11-07 01:46:50]

lgtm

[lgarron, 2015-11-07 01:51:33]

lgarron@chromium.org changed reviewers:
+ pfeldman@chromium.org, pkasting@chromium.org

[lgarron, 2015-11-07 01:51:34]

pkasting@, could you review chrome/browser/ui (2 files)?
pfeldman@, could you review the other two files?

[Peter Kasting, 2015-11-07 02:33:18]

LGTM

https://codereview.chromium.org/1358063002/diff/1/chrome/browser/ui/browser.cc
File chrome/browser/ui/browser.cc (right):

https://codereview.chromium.org/1358063002/diff/1/chrome/browser/ui/browser.c...
chrome/browser/ui/browser.cc:1385: if
(net::IsCertStatusMinorError(security_info.cert_status))
Nit: This now needs {}

[pfeldman, 2015-11-09 23:01:33]

devtools lgtm

[estark, 2015-11-18 22:09:51]

Hey Lucas are you waiting on anything to land this?

[lgarron, 2015-11-18 22:11:35]

On 2015/11/18 at 22:09:51, estark wrote:
> Hey Lucas are you waiting on anything to land this?

I was waiting for branch in case this caused any merge/rebase problems. I'll
land it now.

[lgarron, 2015-11-18 22:11:41]

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

[commit-bot: I haz the power, 2015-11-18 22:13:02]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1358063002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1358063002/1

[commit-bot: I haz the power, 2015-11-18 22:16:10]

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

[commit-bot: I haz the power, 2015-11-18 22:16:11]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[lgarron, 2015-11-18 23:18:38]

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

[commit-bot: I haz the power, 2015-11-18 23:19:58]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1358063002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1358063002/20001

[commit-bot: I haz the power, 2015-11-19 01:32:37]

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

[commit-bot: I haz the power, 2015-11-19 01:32:38]

Dry run: This issue passed the CQ dry run.

[lgarron, 2015-11-19 01:32:59]

The CQ bit was checked by lgarron@chromium.org

[lgarron, 2015-11-19 01:32:59]

The patchset sent to the CQ was uploaded after l-g-t-m from
pkasting@chromium.org, pfeldman@chromium.org, estark@chromium.org
Link to the patchset: https://codereview.chromium.org/1358063002/#ps20001
(title: "Rebase.")

[commit-bot: I haz the power, 2015-11-19 01:40:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1358063002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1358063002/20001

[commit-bot: I haz the power, 2015-11-19 02:20:33]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-19 02:21:34]

Patchset 2 (id:??) landed as
https://crrev.com/8555a6c0f5a20b3409de73b7b120d5701b4668b2
Cr-Commit-Position: refs/heads/master@{#360489}