Issue metadata
Sign in to add a comment
|
Merge SecurityStyleExplanation lists into a single list. |
||||||||||||||||||||||
Issue description
{secure, neutral, insecure, info}_explanations are displayed in DevTools in groups, starting with insecure.
Issue 736183 will cause non-secure mixed content explanations in DevTools to be shown at the top of the explanations. I think it was better to keep the mixed content explanations at the bottom deterministically.
In any case, storing all the explanations in a single list gives us flexibility to control the order.
,
Jul 7 2017
Note: the effect here is that bullets would be shown in the order they are added. This means that we can add explanations in content_utils.cc in the same order every time (e.g. certificate, connection, subresource security), and have them show up in that order. Since we never ended up treating the sever explanations specially while showing them at the top (e.g. using a line to tie them to the lock icon like in earlier mocks), I think the visual cue of predictable order outweighs moving the most severe issues to the top.
,
Jul 7 2017
,
Jul 7 2017
,
Nov 10 2017
,
Feb 18 2018
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by lgar...@chromium.org
, Jul 7 2017