UI review did not approved the chip yet so it must be hidden by default. The EV cert must always be shown, even when the "Security-Chip" flag is set to "Show Non-secure Only"
This needs to be in for the M55 beta
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/eb99caf19d7d759e76b07a00f224c88571e30e4c commit eb99caf19d7d759e76b07a00f224c88571e30e4c Author: spqchan <spqchan@chromium.org> Date: Wed Oct 19 20:31:06 2016 [Views][Mac] Security Chip Default Value - Hide the chip by default - Show the EV certificate for all flag option BUG= 657498 Review-Url: https://chromiumcodereview.appspot.com/2435493003 Cr-Commit-Position: refs/heads/master@{#426270} [modify] https://crrev.com/eb99caf19d7d759e76b07a00f224c88571e30e4c/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.h [modify] https://crrev.com/eb99caf19d7d759e76b07a00f224c88571e30e4c/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm [modify] https://crrev.com/eb99caf19d7d759e76b07a00f224c88571e30e4c/chrome/browser/ui/views/location_bar/location_bar_view.cc [modify] https://crrev.com/eb99caf19d7d759e76b07a00f224c88571e30e4c/chrome/browser/ui/views/location_bar/location_bar_view.h
Your change meets the bar and is auto-approved for M55 (branch: 2883)
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/500e806145aed7d9ff42826ec67f73f66fe7e50a commit 500e806145aed7d9ff42826ec67f73f66fe7e50a Author: spqchan <spqchan@chromium.org> Date: Thu Oct 20 21:05:11 2016 [Views][Mac] Security Chip Default Value - Hide the chip by default - Show the EV certificate for all flag option BUG= 657498 Review-Url: https://chromiumcodereview.appspot.com/2435493003 Cr-Commit-Position: refs/heads/master@{#426270} (cherry picked from commit eb99caf19d7d759e76b07a00f224c88571e30e4c) Review URL: https://codereview.chromium.org/2431223007 . Cr-Commit-Position: refs/branch-heads/2883@{#221} Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768} [modify] https://crrev.com/500e806145aed7d9ff42826ec67f73f66fe7e50a/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.h [modify] https://crrev.com/500e806145aed7d9ff42826ec67f73f66fe7e50a/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm [modify] https://crrev.com/500e806145aed7d9ff42826ec67f73f66fe7e50a/chrome/browser/ui/views/location_bar/location_bar_view.cc [modify] https://crrev.com/500e806145aed7d9ff42826ec67f73f66fe7e50a/chrome/browser/ui/views/location_bar/location_bar_view.h
[Automated comment] removing mislabelled merge-merged-2840
Comment 1 by spqc...@chromium.org
, Oct 19 2016