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

Unified Diff: chrome/browser/printing/print_system_task_proxy.cc

Issue 10545181: Optimizes loading initial printer on non-chromeos platforms. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Add bug reference to TODOs. Created 8 years, 6 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
« no previous file with comments | « no previous file | chrome/browser/resources/print_preview/data/destination_store.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/printing/print_system_task_proxy.cc
diff --git a/chrome/browser/printing/print_system_task_proxy.cc b/chrome/browser/printing/print_system_task_proxy.cc
index 38c6c26a83064a16c4c9933b335c3ec65614bcde..51b96cf68452355a9cbfe336e20c1021d1b4500e 100644
--- a/chrome/browser/printing/print_system_task_proxy.cc
+++ b/chrome/browser/printing/print_system_task_proxy.cc
@@ -117,6 +117,7 @@ using content::BrowserThread;
namespace {
+const char kPrinterId[] = "printerId";
const char kDisableColorOption[] = "disableColorOption";
const char kSetColorAsDefault[] = "setColorAsDefault";
const char kSetDuplexAsDefault[] = "setDuplexAsDefault";
@@ -559,6 +560,7 @@ void PrintSystemTaskProxy::GetPrinterCapabilities(
printer_color_space_for_black));
DictionaryValue settings_info;
+ settings_info.SetString(kPrinterId, printer_name);
settings_info.SetBoolean(kDisableColorOption, disable_color_options);
if (printer_color_space_for_color == printing::UNKNOWN_COLOR_MODEL)
printer_color_space_for_color = printing::COLOR;
« no previous file with comments | « no previous file | chrome/browser/resources/print_preview/data/destination_store.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698