New issue
Advanced search Search tips

Issue 856382 link

Starred by 1 user
Status: Fixed
Owner:
luum@chromium.org
Closed: Jul 10
Cc:
skau@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Task



Sign in to add a comment

Update Docs for CupsPrintersManager

Project Member Reported by valleau@chromium.org, Jun 25 2018 
The documentation for CupsPrintersManager is out of date as the class has been updated to be a BrowserContextKeyedService.

Update the documentation to reflect this.

Link to document:
https://chromium.googlesource.com/chromium/src/+/06ed7efbb6a94ce8019f269a90dba3124c7d730c/chrome/browser/chromeos/printing/doc/cups_printer_management.md
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 10 

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

commit 78a1d90aa9ba07115b72462cc1733fb911efd57c
Author: Luum Habtemariam <luum@chromium.org>
Date: Tue Jul 10 21:25:48 2018

Updating CupsPrintersManager docs

Recently changed CupsPrintersManager into BrowserContextKeyedService.
This change reflects that in the documentation.

Bug:  chromium:856382 
Test: Manually checked doc page
Change-Id: I8fc9859beea1776814b342f0cf4633d2ff8a2817
Reviewed-on: https://chromium-review.googlesource.com/1117267
Commit-Queue: Luum Habtemariam <luum@chromium.org>
Reviewed-by: Sean Kau <skau@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#573915}
[modify] https://crrev.com/78a1d90aa9ba07115b72462cc1733fb911efd57c/chrome/browser/chromeos/printing/doc/cups_printer_management.md

Comment 2 by luum@chromium.org, Jul 10

Status: Fixed (was: Assigned)
►

Sign in to add a comment