New issue
Advanced search Search tips

Issue 637272 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: 2016-10-21
OS: Chrome
Pri: 1
Type: Feature

Blocked on:
issue 624920
issue 658461
issue 658463



Sign in to add a comment

Integration of print dialog UI with backend

Project Member Reported by weifangsun@chromium.org, Aug 12 2016

Issue description

Integrate calls to backend for population/action items on print dialog in the UI
 
Blocking: 624920
Blockedon: 624920
Blocking: -624920
Labels: Week-1637

Comment 4 by skau@chromium.org, Oct 3 2016

Starting this week 10/3
Labels: -Week-1637 Week-1641

Comment 6 by skau@chromium.org, Oct 14 2016

NextAction: 2016-10-21
I've been busy with sync and policy.  No update.

Comment 7 by skau@chromium.org, Oct 21 2016

Status: Started (was: Assigned)
Drafting design/reverse engineering current design 10/21

Comment 8 by skau@chromium.org, Oct 21 2016

Blockedon: 658461

Comment 9 by skau@chromium.org, Oct 22 2016

Blockedon: 658463

Comment 10 by adlr@chromium.org, Nov 4 2016

How does this bug differ from https://bugs.chromium.org/p/chromium/issues/detail?id=658461 ?

Comment 11 by skau@chromium.org, Nov 4 2016

This bug connects all the parts together.   crbug.com/658461  is part 1 of 3.
Project Member

Comment 12 by bugdroid1@chromium.org, Nov 5 2016

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

commit 7fb79ad447b89800ca047c2de1b59096231bae4d
Author: skau <skau@chromium.org>
Date: Sat Nov 05 01:25:21 2016

Use printer id to populate CUPS instead of printer name.

Printer name can be ambiguous.  Printer id is not ambiguous.  Register
printers using printer id instead of printer name using CUPS and fix
some resulting problems.

CL 2 of 3 to enable printer selection on Chrome OS

BUG= 637272 

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

[modify] https://crrev.com/7fb79ad447b89800ca047c2de1b59096231bae4d/chrome/browser/ui/webui/settings/chromeos/cups_printers_handler.cc
[modify] https://crrev.com/7fb79ad447b89800ca047c2de1b59096231bae4d/chromeos/printing/printer_configuration.cc

Comment 13 by skau@chromium.org, Nov 9 2016

Status: Fixed (was: Started)

Comment 14 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58

Comment 15 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

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

Labels: VerifyIn-60
Labels: VerifyIn-61

Comment 18 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)

Sign in to add a comment