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

Issue 658928 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Feature

Blocking:
issue 626752



Sign in to add a comment

Implement UI to link Printer Setup from existing Settings to MD Settings

Project Member Reported by weifangsun@chromium.org, Oct 24 2016

Issue description

The CUPS printer setup UI is currently implemented under MD Settings.

Because we are planning to launch CUPS with M56 and CrOS MD Settings will launch in a following milestone, we need a UI solution for accessing the CUPS printer setup UI from the existing settings.

 
Cc: -x...@chromium.org jonnymack@chromium.org
Owner: x...@chromium.org
Proposed UX: https://docs.google.com/presentation/d/1z9tntdvKHNloLrs2zyjLWeNzo9eDl6qVUA3e1Ddfjwo/edit#slide=id.g187cb7866e_0_19

Comment 2 by x...@chromium.org, Oct 26 2016

Blocking: 626752

Comment 3 by x...@chromium.org, Nov 3 2016

https://codereview.chromium.org/2464403003/ is under review.
Project Member

Comment 4 by bugdroid1@chromium.org, Nov 4 2016

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

commit b0e2e70344284572abfdf5c5dc2aa15fc3a32559
Author: xdai <xdai@chromium.org>
Date: Fri Nov 04 22:37:46 2016

[CUPS] Implement the "Printers" section in chrome://settings.

The CUPS printer setup UI is currently implemented under MD Settings.

Because we are planning to launch CUPS with M56 and CrOS MD Settings
will launch in a following milestone, we implement the "Printers" section
in current chrome://settings to link to the CUPS printer MD settings UI
from the existing chrome://settings.

Note: This change is temporary and will be reverted later in M57.

BUG= 658928 , 626752
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2464403003
Cr-Commit-Position: refs/heads/master@{#430039}

[modify] https://crrev.com/b0e2e70344284572abfdf5c5dc2aa15fc3a32559/chrome/app/generated_resources.grd
[modify] https://crrev.com/b0e2e70344284572abfdf5c5dc2aa15fc3a32559/chrome/browser/resources/options/browser_options.html
[modify] https://crrev.com/b0e2e70344284572abfdf5c5dc2aa15fc3a32559/chrome/browser/resources/options/browser_options.js
[modify] https://crrev.com/b0e2e70344284572abfdf5c5dc2aa15fc3a32559/chrome/browser/ui/webui/options/browser_options_handler.cc
[modify] https://crrev.com/b0e2e70344284572abfdf5c5dc2aa15fc3a32559/chrome/browser/ui/webui/options/browser_options_handler.h
[modify] https://crrev.com/b0e2e70344284572abfdf5c5dc2aa15fc3a32559/chrome/browser/ui/webui/print_preview/print_preview_ui.cc
[modify] https://crrev.com/b0e2e70344284572abfdf5c5dc2aa15fc3a32559/chrome/common/url_constants.cc

Comment 5 by x...@chromium.org, Jan 5 2017

Status: Fixed (was: Assigned)

Comment 6 by dchan@google.com, May 30 2017

Labels: VerifyIn-60

Comment 7 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

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

Status: Archived (was: Fixed)

Sign in to add a comment