New issue
Advanced search Search tips

Issue 717802 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Nov 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug
Team-Security-UX

Blocking:
issue 663975


Show other hotlists

Hotlists containing this issue:
EnamelAndFriendsFixIt


Sign in to add a comment

#ifdef IDS_CERT_POLICY_PROVIDED_CERT_MESSAGE to be used on Chrome OS only

Project Member Reported by lgar...@chromium.org, May 3 2017

Issue description

#ifdef in page_info.cc
<if expr="chromium"> in page_info_strings.grd
 
Labels: Enterprise-Traged
Owner: lgar...@chromium.org
Status: Assigned (was: Available)

Comment 3 by est...@chromium.org, Nov 10 2017

Labels: Hotlist-EnamelAndFriendsFixIt
Owner: elawrence@chromium.org
Project Member

Comment 5 by bugdroid1@chromium.org, Nov 21 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6758d1d985a661a36c0cfacbc951ccc3d3d22fc3

commit 6758d1d985a661a36c0cfacbc951ccc3d3d22fc3
Author: Eric Lawrence <elawrence@chromium.org>
Date: Tue Nov 21 16:06:05 2017

Omit policy-installed certificate message on unsupported platforms

Only ChromeOS supports policy-installed certificates. Omit the string
describing this state from the PageInfo UI's resources except when
compiling for ChromeOS.

Bug:  717802 
Change-Id: I106b03b8cac1a15e3937996585185e2e26279d4a
Reviewed-on: https://chromium-review.googlesource.com/779880
Reviewed-by: Emily Stark <estark@chromium.org>
Commit-Queue: Eric Lawrence <elawrence@chromium.org>
Cr-Commit-Position: refs/heads/master@{#518267}
[modify] https://crrev.com/6758d1d985a661a36c0cfacbc951ccc3d3d22fc3/chrome/browser/ui/page_info/page_info.cc
[modify] https://crrev.com/6758d1d985a661a36c0cfacbc951ccc3d3d22fc3/chrome/browser/ui/page_info/page_info_unittest.cc
[modify] https://crrev.com/6758d1d985a661a36c0cfacbc951ccc3d3d22fc3/components/page_info_strings.grdp

Status: Fixed (was: Assigned)

Comment 7 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Comment 8 by dchan@chromium.org, Jan 23 2018

Status: Fixed (was: Archived)
Status: Verified (was: Fixed)
As verified in the code review.

Sign in to add a comment