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

Side by Side Diff: printing/printing_context_no_system_dialog.cc

Issue 16915006: Convert most of extensions and some other random stuff to using the base namespace for Values. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "printing/printing_context_no_system_dialog.h" 5 #include "printing/printing_context_no_system_dialog.h"
6 6
7 #include <unicode/ulocdata.h> 7 #include <unicode/ulocdata.h>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/values.h" 10 #include "base/values.h"
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
72 72
73 settings_.set_dpi(dpi); 73 settings_.set_dpi(dpi);
74 settings_.SetPrinterPrintableArea(physical_size_device_units, 74 settings_.SetPrinterPrintableArea(physical_size_device_units,
75 printable_area_device_units, 75 printable_area_device_units,
76 dpi); 76 dpi);
77 77
78 return OK; 78 return OK;
79 } 79 }
80 80
81 PrintingContext::Result PrintingContextNoSystemDialog::UpdatePrinterSettings( 81 PrintingContext::Result PrintingContextNoSystemDialog::UpdatePrinterSettings(
82 const DictionaryValue& job_settings, const PageRanges& ranges) { 82 const base::DictionaryValue& job_settings, const PageRanges& ranges) {
83 bool landscape = false; 83 bool landscape = false;
84 84
85 if (!job_settings.GetBoolean(kSettingLandscape, &landscape)) 85 if (!job_settings.GetBoolean(kSettingLandscape, &landscape))
86 return OnError(); 86 return OnError();
87 87
88 if (settings_.dpi() == 0) 88 if (settings_.dpi() == 0)
89 UseDefaultSettings(); 89 UseDefaultSettings();
90 90
91 settings_.SetOrientation(landscape); 91 settings_.SetOrientation(landscape);
92 settings_.ranges = ranges; 92 settings_.ranges = ranges;
(...skipping 56 matching lines...) Expand 10 before | Expand all | Expand 10 after
149 // Intentional No-op. 149 // Intentional No-op.
150 } 150 }
151 151
152 gfx::NativeDrawingContext PrintingContextNoSystemDialog::context() const { 152 gfx::NativeDrawingContext PrintingContextNoSystemDialog::context() const {
153 // Intentional No-op. 153 // Intentional No-op.
154 return NULL; 154 return NULL;
155 } 155 }
156 156
157 } // namespace printing 157 } // namespace printing
158 158
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698