Add an existence check to synced printers manager |
|||
Issue descriptionWe're currently unmarshalling a printer object just to check if we've already saved it. Add a method that just returns true/false based on the id.
,
Feb 15 2018
,
Feb 23 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6459a2fddb51db29fd11e908d0c4cb19097a4320 commit 6459a2fddb51db29fd11e908d0c4cb19097a4320 Author: David Valleau <valleau@chromium.org> Date: Fri Feb 23 01:11:52 2018 Adding a "IsPrinterRegistered" Function to SyncedPrintersManager R=skau@chromium.org Bug: 777035 Change-Id: Ia33b81c1fe77b8b373fd16b91693e18287782fef Reviewed-on: https://chromium-review.googlesource.com/929873 Reviewed-by: Sean Kau <skau@chromium.org> Commit-Queue: Sean Kau <skau@chromium.org> Cr-Commit-Position: refs/heads/master@{#538651} [modify] https://crrev.com/6459a2fddb51db29fd11e908d0c4cb19097a4320/chrome/browser/chromeos/printing/printers_sync_bridge.cc [modify] https://crrev.com/6459a2fddb51db29fd11e908d0c4cb19097a4320/chrome/browser/chromeos/printing/printers_sync_bridge.h [modify] https://crrev.com/6459a2fddb51db29fd11e908d0c4cb19097a4320/chrome/browser/chromeos/printing/synced_printers_manager.cc
,
Feb 23 2018
|
|||
►
Sign in to add a comment |
|||
Comment 1 by skau@chromium.org
, Feb 15 2018