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

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

Issue 7346017: Clean up users of a deprecated base::LaunchApp API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: cros Created 9 years, 5 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/printing/printer_manager_dialog_linux.cc
diff --git a/chrome/browser/printing/printer_manager_dialog_linux.cc b/chrome/browser/printing/printer_manager_dialog_linux.cc
index 61cea9d9e1e8c20bafc0c02883b5451003d567f6..02b262c80a9f9886e7f85ddd4000044357940309 100644
--- a/chrome/browser/printing/printer_manager_dialog_linux.cc
+++ b/chrome/browser/printing/printer_manager_dialog_linux.cc
@@ -47,9 +47,10 @@ void DetectAndOpenPrinterConfigDialog() {
std::vector<std::string> argv;
argv.push_back(command);
- base::file_handle_mapping_vector no_files;
+ base::LaunchOptions options;
base::ProcessHandle handle;
- if (!base::LaunchApp(argv, no_files, false, &handle)) {
+ options.process_handle = &handle;
+ if (!base::LaunchProcess(argv, options)) {
LOG(ERROR) << "Failed to open printer manager dialog ";
return;
}

Powered by Google App Engine
This is Rietveld 408576698