New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 785242 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 29
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Feature



Sign in to add a comment

Disable Header Footer Url through policy

Reported by francois...@gmail.com, Nov 15 2017

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/62.0.3202.94 Safari/537.36

Steps to reproduce the problem:
1. Trying to disable header footer url
2. 
3. 

What is the expected behavior?
Disable header footer url when printing for all of our Citrix users

What went wrong?
We had an organization's request to disable the header url when priting. How can we set it since the parameter is not avaible through GPO? We try to set it through the master_preference file with the {\"isHeaderFooterEnabled\":false} parameter but it doesnt work. Any suggestions?

Did this work before? No 

Chrome version:  62.0.3202.62  Channel: stable
OS Version: 10.0
Flash Version:
 
Cc: kkaluri@chromium.org
Kiran, can you please take a look?
Cc: georgesak@chromium.org blumberg@chromium.org
Labels: Needs-Feedback
francois.guicheman@ Could you confirm whether this issue is used to work before or this is a feature request ?
We didnt manage to get this work and we dont know if its possible or not.

We just want to know if its possible to disable header footer url by GPO or by chrome configuration for all users.

Regards.
Project Member

Comment 5 by sheriffbot@chromium.org, Nov 17 2017

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "kkaluri@chromium.org" to the cc list and removing "Needs-Feedback" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: -georgesak@chromium.org
Labels: -Type-Bug Enterprise-Triaged Enterprise-Policy Type-Feature
Owner: georgesak@chromium.org
Status: Available (was: Unconfirmed)
Summary: Disable Header Footer Url through policy (was: Disable Header Footer Url )
It's currently not possible.

Changing this to FR instead of bug.
Ok thanks for your reply. Do you know if this will be supported in a future release?
Components: UI>Browser>PrintPreview
This is related to  bug 759978 . Would just disabling headers and footer altogether work for you? I'm also curious why folks using Citrix want to do this. Sensitive info in the URLs or they are just garbage data?

BTW, "isHeaderFooterEnabled" is not a setting in the master preferences file.
How can we do this since there is no such parameter related to header/footer in GPO? (Ofc its work if you directly uncheck the box in your browser before printing but thats not what users want..)

Yeah kind of both, sensitive in some case because its medical stuff but most of the time garbage data...
We definitely want to work with you to figure out what it is you really need. e.g. Policies and master preferences do different things. See: https://www.chromium.org/administrators/configuring-other-preferences

Is the goal to set header/footers off by default or to force it off?
Hi,

one of our customers wants to specify print settings (margins and turning off header/footer). It would be sufficient to deploy default settings via GPO (or master prefs), as the users shall be able to use their desired settings!

Sorry for piggybacking this thread, but as it has not been updated anymore by Francois, I thought this might be ok!
Sorry for the delay.

The goal is to force it off, not to let the user choose it. Since we are though Citrix, we didnt save the option in the user profil (roaming profil) because we dont know where this settings is setted. 
Cc: pfeldman@chromium.org
We are about add custom header/footer styling support. The first user is via headless mode, to allow Puppeteer (https://github.com/GoogleChrome/puppeteer) and potentially other headless clients to customize their save to PDF output.

Given this mechanism will soon be available, maybe that's what we should expose as an enterprise policy. I believe it is flexable enough to support the use case here.
For reference, r523674 changed the way the header/footer in Print Preview is constructed, and https://crrev.com/c/809759 will allow control.
Hello,

Thanks for the reply, yes it would be really appreciated that enterprise could customize this setting by your admx composent or a least a way to do it for all users.

Do you have any idea of the timing?
The Chrome Enterprise folks need to make a call on adding the policy. If we added the policy today, per https://www.chromium.org/developers/calendar, it will release users on Stable Channel in March.
That would be really great :)
This policy request is in our pipeline now. Hopefully we can work on it soon. We'll update this bug once we do.
Cc: georgesak@chromium.org
Owner: nicolaso@chromium.org
Status: Assigned (was: Available)
Project Member

Comment 21 by bugdroid1@chromium.org, Aug 6

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

commit 4dbdf1e51445d45d977685c6280fbcdbbe046aa9
Author: Nicolas Ouellet-Payeur <nicolaso@chromium.org>
Date: Mon Aug 06 21:51:02 2018

Add a pref to force "Headers and Footers" in print preview

This is the first step towards creating a policy to enforce printing
with/without headers and footers. When it is set, the 'Headers and
Footers' checkbox is disabled and greyed out.

Right now, there is no easy way to set the pref (beyond just editing the
Preferences JSON file).

Screenshot:
https://screenshot.googleplex.com/LWBrvFzkRZ5.png

Bug:  785242 
Change-Id: I4a8bb8a0ec5455fe8467931f27eb5e6fab43dd2e
Reviewed-on: https://chromium-review.googlesource.com/1149994
Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Commit-Queue: Nicolas Ouellet-Payeur <nicolaso@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580999}
[modify] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/browser/prefs/browser_prefs.cc
[modify] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/browser/resources/print_preview/native_layer.js
[modify] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/browser/resources/print_preview/new/app.js
[modify] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/browser/resources/print_preview/new/model.js
[modify] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/browser/resources/print_preview/new/other_options_settings.html
[modify] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/browser/resources/print_preview/new/other_options_settings.js
[modify] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/browser/resources/print_preview/new/settings_behavior.js
[modify] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/browser/ui/BUILD.gn
[add] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/browser/ui/webui/print_preview/policy_settings.cc
[add] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/browser/ui/webui/print_preview/policy_settings.h
[modify] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/browser/ui/webui/print_preview/print_preview_handler.cc
[modify] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/browser/ui/webui/print_preview/print_preview_handler.h
[modify] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/browser/ui/webui/print_preview/print_preview_handler_unittest.cc
[modify] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/common/pref_names.cc
[modify] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/common/pref_names.h
[modify] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/test/data/webui/print_preview/model_test.js
[modify] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/test/data/webui/print_preview/new_print_preview_ui_browsertest.js
[modify] https://crrev.com/4dbdf1e51445d45d977685c6280fbcdbbe046aa9/chrome/test/data/webui/print_preview/settings_section_test.js

Project Member

Comment 22 by bugdroid1@chromium.org, Aug 29

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

commit de05f8f9dc649b306c6ba098c45977ed84894960
Author: Nicolas Ouellet-Payeur <nicolaso@chromium.org>
Date: Wed Aug 29 19:41:37 2018

Add PrintHeaderFooter policy and map to printing.print_header_footer

This is a follow-up to:
https://chromium-review.googlesource.com/c/chromium/src/+/1149994

Bug:  785242 
Change-Id: Iddd3019d7b6695c4ff6f03364fb9c72a1afee446
Reviewed-on: https://chromium-review.googlesource.com/1170855
Commit-Queue: Nicolas Ouellet-Payeur <nicolaso@chromium.org>
Reviewed-by: Julian Pastarmov <pastarmovj@chromium.org>
Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Reviewed-by: Owen Min <zmin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#587249}
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/chrome/browser/policy/configuration_policy_handler_list_factory.cc
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/chrome/browser/resources/print_preview/native_layer.js
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/chrome/browser/resources/print_preview/new/app.js
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/chrome/browser/resources/print_preview/new/model.js
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/chrome/browser/ui/webui/print_preview/policy_settings.cc
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/chrome/browser/ui/webui/print_preview/policy_settings.h
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/chrome/browser/ui/webui/print_preview/print_preview_handler.cc
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/chrome/browser/ui/webui/print_preview/print_preview_handler.h
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/chrome/browser/ui/webui/print_preview/print_preview_handler_unittest.cc
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/chrome/common/pref_names.cc
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/chrome/test/data/policy/policy_test_cases.json
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/chrome/test/data/webui/print_preview/model_test.js
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/chrome/test/data/webui/print_preview/new_print_preview_ui_browsertest.js
[add] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/chrome/test/data/webui/print_preview/policy_test.js
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/chrome/test/data/webui/print_preview/print_preview_test_utils.js
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/chrome/test/data/webui/print_preview/settings_section_test.js
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/components/policy/resources/policy_templates.json
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/testing/buildbot/filters/webui_polymer2_browser_tests.filter
[modify] https://crrev.com/de05f8f9dc649b306c6ba098c45977ed84894960/tools/metrics/histograms/enums.xml

Status: Fixed (was: Assigned)
Cc: pastarmovj@chromium.org ranjitkan@chromium.org abdulsyed@chromium.org ligim...@chromium.org pbomm...@chromium.org bustamante@chromium.org gov...@chromium.org nicolaso@chromium.org
 Issue 759978  has been merged into this issue.

Sign in to add a comment