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

Issue 724278 link

Starred by 3 users

Issue metadata

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



Sign in to add a comment

Crashes after click 'Add printer' in chrome://md-settings/cupsPrinters

Project Member Reported by xiaochu@chromium.org, May 18 2017

Issue description

I'm build go/simplechrome on top of tree and deploy to a chromebook.
 
Components: Internals>Printing>CUPS

Comment 2 by ka...@chromium.org, May 18 2017

Cc: skau@chromium.org adlr@chromium.org

Comment 3 by ka...@chromium.org, May 18 2017

Labels: OS-Chrome

Comment 4 by skau@chromium.org, May 22 2017

Problem occurs in guest mode.

Comment 5 by skau@chromium.org, May 22 2017

Owner: justincarlson@chromium.org

Comment 6 by skau@chromium.org, May 22 2017

Labels: -Pri-3 Pri-1

Comment 7 by skau@chromium.org, May 22 2017

Cc: justincarlson@chromium.org
Owner: skau@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 8 by bugdroid1@chromium.org, May 23 2017

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

commit cb6535121b738ffffecab30176010bb03e5ca447
Author: skau <skau@chromium.org>
Date: Tue May 23 00:30:03 2017

Allow the PrinterDetector to be constructed with a delegated profile.

The add printer dialog was crashing in Guest Mode because a
PrinterDetector could not be constructed.  It is okay for
PrinterDetector to be shared between logged in profiles.

BUG= 724278 

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

[modify] https://crrev.com/cb6535121b738ffffecab30176010bb03e5ca447/chrome/browser/chromeos/printer_detector/printer_detector_factory.cc
[modify] https://crrev.com/cb6535121b738ffffecab30176010bb03e5ca447/chrome/browser/chromeos/printer_detector/printer_detector_factory.h

Comment 9 by skau@chromium.org, May 23 2017

Status: Fixed (was: Started)

Comment 10 by skau@chromium.org, May 24 2017

Cc: steve...@chromium.org pucchakayala@chromium.org songsuk@chromium.org ajha@chromium.org kavvaru@chromium.org brajkumar@chromium.org
 Issue 725445  has been merged into this issue.
Labels: VerifyIn-61

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

Status: Archived (was: Fixed)

Sign in to add a comment