New issue
Advanced search Search tips

Issue 804322 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Mar 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Remove the product messaging from chrome://policy

Project Member Reported by atwilson@chromium.org, Jan 22 2018

Issue description

The about:policy page has the following link: "Using Chrome at work? Businesses can manage Chrome settings for their employees. Learn more"

This is just weird on a number of counts:

1) about://policy is a diagnostic page and not a good place to put product messaging.
2) the link is shown even if the user already *has* policy
3) the person who is likely to see this is an end user, not an admin.

So let's remove this link (basically, revert https://codereview.chromium.org/2386243002). I'm keeping this on my plate as a GoodFirstBug to give to nooglers.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 16 2018

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

commit 3de50d0eebb49a9c54a962798814c9e2f8eb704b
Author: Ganesh Borle <ganesh.borle@samsung.com>
Date: Fri Feb 16 05:30:59 2018

Removed the product message from chrome://policy

Bug:  804322 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ie114ad5c38fbef3cd32b3f830eaa663596041c9d
Reviewed-on: https://chromium-review.googlesource.com/908668
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Reviewed-by: Drew Wilson <atwilson@chromium.org>
Commit-Queue: srirama chandra sekhar <srirama.m@samsung.com>
Cr-Commit-Position: refs/heads/master@{#537215}
[modify] https://crrev.com/3de50d0eebb49a9c54a962798814c9e2f8eb704b/chrome/browser/resources/policy.css
[modify] https://crrev.com/3de50d0eebb49a9c54a962798814c9e2f8eb704b/chrome/browser/resources/policy.html
[modify] https://crrev.com/3de50d0eebb49a9c54a962798814c9e2f8eb704b/chrome/browser/ui/webui/policy_ui.cc
[modify] https://crrev.com/3de50d0eebb49a9c54a962798814c9e2f8eb704b/components/policy_strings.grdp

Status: Verified (was: Untriaged)

Sign in to add a comment