Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(30)

Unified Diff: chromeos/printing/ppd_cache.cc

Issue 2545663002: Hookup the PpdProvider in the printer setup flow. (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chromeos/printing/ppd_cache.cc
diff --git a/chromeos/printing/ppd_cache.cc b/chromeos/printing/ppd_cache.cc
index 2726b8ac95ecf3f678fd3c76d9fbd02ca14a039e..2f8b027652cf98990b62d8fed923b876218570e1 100644
--- a/chromeos/printing/ppd_cache.cc
+++ b/chromeos/printing/ppd_cache.cc
@@ -68,6 +68,7 @@ class PpdCacheImpl : public PpdCache {
base::Optional<base::FilePath> Store(
const Printer::PpdReference& reference,
const std::string& ppd_contents) override {
+ VLOG(1) << "Storing ppd";
Carlson 2016/12/01 01:58:44 Reminder to remove this logging.
skau 2016/12/01 21:45:12 Done.
base::ThreadRestrictions::AssertIOAllowed();
if (!EnsureCacheDirectoryExists()) {
return base::nullopt;
@@ -76,11 +77,13 @@ class PpdCacheImpl : public PpdCache {
base::FilePath contents_path =
GetCachePathBase(reference).AddExtension(".ppd");
if (IsGZipped(ppd_contents)) {
+ VLOG(1) << "PPD is gzipped";
contents_path = contents_path.AddExtension(".gz");
}
if (base::WriteFile(contents_path, ppd_contents.data(),
ppd_contents.size()) ==
static_cast<int>(ppd_contents.size())) {
+ VLOG(1) << "Stored at path: " << contents_path.value();
ret = contents_path;
} else {
LOG(ERROR) << "Failed to write " << contents_path.LossyDisplayName();

Powered by Google App Engine
This is Rietveld 408576698