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

Unified Diff: chrome/browser/download/save_package_file_picker_chromeos.cc

Issue 12334073: Remove WebContents methods that duplicate WebContentsView methods. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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/download/save_package_file_picker_chromeos.cc
diff --git a/chrome/browser/download/save_package_file_picker_chromeos.cc b/chrome/browser/download/save_package_file_picker_chromeos.cc
index 0ab46920484920ba0355fe525214ec1c9c1d0eb0..f9f8ab11eef504795d498440f5e2c513f7639534 100644
--- a/chrome/browser/download/save_package_file_picker_chromeos.cc
+++ b/chrome/browser/download/save_package_file_picker_chromeos.cc
@@ -15,6 +15,7 @@
#include "chrome/browser/ui/chrome_select_file_policy.h"
#include "content/public/browser/download_item.h"
#include "content/public/browser/web_contents.h"
+#include "content/public/browser/web_contents_view.h"
#include "ui/shell_dialogs/selected_file_info.h"
namespace {
@@ -53,15 +54,15 @@ SavePackageFilePickerChromeOS::SavePackageFilePickerChromeOS(
this, new ChromeSelectFilePolicy(web_contents));
ui::SelectFileDialog::FileTypeInfo file_types;
file_types.support_drive = true;
- select_file_dialog_->SelectFile(ui::SelectFileDialog::SELECT_SAVEAS_FILE,
- string16(),
- suggested_path.ReplaceExtension("mhtml"),
- &file_types,
- 0,
- "mhtml",
- platform_util::GetTopLevel(
- web_contents->GetNativeView()),
- NULL);
+ select_file_dialog_->SelectFile(
+ ui::SelectFileDialog::SELECT_SAVEAS_FILE,
+ string16(),
+ suggested_path.ReplaceExtension("mhtml"),
+ &file_types,
+ 0,
+ "mhtml",
+ platform_util::GetTopLevel(web_contents->GetView()->GetNativeView()),
+ NULL);
} else {
FileSelected(suggested_path.ReplaceExtension("mhtml"), 0, NULL);
}

Powered by Google App Engine
This is Rietveld 408576698