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

Unified Diff: chrome/browser/ui/webui/print_preview/extension_printer_handler.cc

Issue 1739183003: Make extensions::DictionaryBuilder and extensions::ListValue unmovable. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 months 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: chrome/browser/ui/webui/print_preview/extension_printer_handler.cc
diff --git a/chrome/browser/ui/webui/print_preview/extension_printer_handler.cc b/chrome/browser/ui/webui/print_preview/extension_printer_handler.cc
index 11b571ec16703c50a5470a4079233a3b0f8ce268..af963e016d207ce1480b4103d7716d0534b83f58 100644
--- a/chrome/browser/ui/webui/print_preview/extension_printer_handler.cc
+++ b/chrome/browser/ui/webui/print_preview/extension_printer_handler.cc
@@ -352,7 +352,7 @@ void ExtensionPrinterHandler::OnUsbDevicesEnumerated(
continue;
}
- printer_list.Append(std::move(
+ printer_list.Append(
DictionaryBuilder()
.Set("id", GenerateProvisionalUsbPrinterId(extension.get(),
device.get()))
@@ -363,7 +363,8 @@ void ExtensionPrinterHandler::OnUsbDevicesEnumerated(
device->product_string(), base::string16(), false))
.Set("extensionId", extension->id())
.Set("extensionName", extension->name())
- .Set("provisional", true)));
+ .Set("provisional", true)
+ .Build());
}
}
}

Powered by Google App Engine
This is Rietveld 408576698