New issue
Advanced search Search tips

Issue 882896 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 13
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Printing policies are not applied

Project Member Reported by vkuzkokov@chromium.org, Sep 11

Issue description

Chrome Version: 71.03550.0 (ToT)
OS: ChromeOS

What steps will reproduce the problem?
(1) Set Printing color policy
(2) Set destination to a non-native destination (e.g. "Save as PDF")
(3) Restart print preview
(4) Switch to a native destination

What is the expected result?
Policy applies (Color drop-down grayed out).

What happens instead?
Nothing
 
Labels: M-70
Don't forget to merge fix in M70, if needed.
Status: Fixed (was: Assigned)
Labels: Merge-Request-70
Labels: Merge-Approved-70
Approved for M70
Labels: -Merge-Request-70
Project Member

Comment 7 by bugdroid1@chromium.org, Sep 17

Labels: -merge-approved-70 merge-merged-3538
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/44a25b17f6513aa30438861e4105d55d9967cceb

commit 44a25b17f6513aa30438861e4105d55d9967cceb
Author: Vladislav Kuzkokov <vkuzkokov@chromium.org>
Date: Mon Sep 17 11:54:28 2018

PrintPreview: Pass policies in "setupPrinter" message.

Bug:  882896 
Change-Id: I21cc0d2fcbfaa7d9f112008fb2612ce6d50f8a8a
Reviewed-on: https://chromium-review.googlesource.com/1219448
Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Commit-Queue: Vladislav Kuzkokov <vkuzkokov@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#590966}(cherry picked from commit a29da1f06f063f246373e30bd9b45d565a0fa554)
Reviewed-on: https://chromium-review.googlesource.com/1227532
Reviewed-by: Vladislav Kuzkokov <vkuzkokov@chromium.org>
Cr-Commit-Position: refs/branch-heads/3538@{#443}
Cr-Branched-From: 79f7c91a2b2a2932cd447fa6f865cb6662fa8fa6-refs/heads/master@{#587811}
[modify] https://crrev.com/44a25b17f6513aa30438861e4105d55d9967cceb/chrome/browser/resources/print_preview/native_layer.js
[modify] https://crrev.com/44a25b17f6513aa30438861e4105d55d9967cceb/chrome/browser/resources/print_preview/new/destination_dialog.js
[modify] https://crrev.com/44a25b17f6513aa30438861e4105d55d9967cceb/chrome/browser/ui/webui/print_preview/print_preview_handler.cc

Project Member

Comment 8 by bugdroid1@chromium.org, Sep 17

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

commit b1cc597fcce990fd672115f90a912b7cc2117c4c
Author: Vladislav Kuzkokov <vkuzkokov@chromium.org>
Date: Mon Sep 17 14:48:55 2018

Add test for "setupPrinter" call that returns policies

Bug:  882896 
Change-Id: Ie0d9098e7f3f801961ed9c6308e453309a39ff40
Reviewed-on: https://chromium-review.googlesource.com/1224531
Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Commit-Queue: Vladislav Kuzkokov <vkuzkokov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#591673}
[modify] https://crrev.com/b1cc597fcce990fd672115f90a912b7cc2117c4c/chrome/test/data/webui/print_preview/destination_search_test.js
[modify] https://crrev.com/b1cc597fcce990fd672115f90a912b7cc2117c4c/chrome/test/data/webui/print_preview/new_print_preview_ui_browsertest.js

Sign in to add a comment