Issue metadata
Sign in to add a comment
|
SECURE_WITH_POLICY_INSTALLED_CERT should show meaningful string in Page Info |
||||||||||||||||||||||
Issue description
Long ago, when the page security state was SECURE_WITH_POLICY_INSTALLED_CERT, PageInfo showed a meaningful string to let the user know why they were seeing a little building icon instead of a lock icon.
The old code is present:
site_identity_status_ = SITE_IDENTITY_STATUS_ADMIN_PROVIDED_CERT;
site_identity_details_ = l10n_util::GetStringFUTF16(
IDS_CERT_POLICY_PROVIDED_CERT_MESSAGE, UTF8ToUTF16(url.host()));
...but I don't think this appears anywhere in a current UX. I think we need a change somewhere around https://cs.chromium.org/chromium/src/chrome/browser/ui/page_info/page_info_ui.cc?l=230&rcl=67fee5924741f4ccd5985b4933a042e8a24559d4
Through various refactors, that string is no longer shown, leaving users
potentially confused about what's going on.
(Note: Long-term, we should get rid of the SECURE_WITH_POLICY_INSTALLED_CERT state entirely and leave it to Chrome OS to notify the user when a policy-cert is installed. But that's a bigger change.)
,
Dec 18 2017
patricialor@, could you help triage this issue?
,
Dec 18 2017
I think this would be best looked at by someone familiar with security states.
,
Jan 17 2018
,
Jan 30 2018
,
Jun 12 2018
Unfortunately, I won't be able to work on this in the foreseeable future.
,
Jun 18 2018
For Drew to assign somebody to work on this.
,
Jun 19 2018
Plan is (I thought) to display session disclosures rather than per-page UX. So effectively we'll do what's suggested as the "long-term plan" in the original report. Thiemo, thinking of duping this against the disclosure bug - WDYT?
,
Jun 20 2018
Agreed that the POR is to remove the enterprise icon and show the green lock icon instead. Independent of that I'd consider a string in the page info dialog useful, though. I think a good case can be made for that string, as it's relevant information that's shown in context, exactly at the time that the user is looking for it. (The string could link to the disclosure view.)
,
Jun 20 2018
Marton, maybe this can be a piece of the disclosure work your intern picks up.
,
Aug 2
,
Oct 25
,
Oct 29
,
Nov 29
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by raymes@chromium.org
, Dec 17 2017