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

Unified Diff: content/browser/web_contents/web_contents_impl.cc

Issue 12213066: Use base namespace for FilePath in content/browser (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: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 3b5ff7cd7f166174131e05e5337754e3d721b1b0..baea232a09b69af87dc9354964773da9a4313863 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -1776,8 +1776,8 @@ void WebContentsImpl::OnSavePage() {
// Used in automated testing to bypass prompting the user for file names.
// Instead, the names and paths are hard coded rather than running them through
// file name sanitation and extension / mime checking.
-bool WebContentsImpl::SavePage(const FilePath& main_file,
- const FilePath& dir_path,
+bool WebContentsImpl::SavePage(const base::FilePath& main_file,
+ const base::FilePath& dir_path,
SavePageType save_type) {
// Stop the page from navigating.
Stop();
@@ -1787,8 +1787,8 @@ bool WebContentsImpl::SavePage(const FilePath& main_file,
}
void WebContentsImpl::GenerateMHTML(
- const FilePath& file,
- const base::Callback<void(const FilePath&, int64)>& callback) {
+ const base::FilePath& file,
+ const base::Callback<void(const base::FilePath&, int64)>& callback) {
MHTMLGenerationManager::GetInstance()->GenerateMHTML(this, file, callback);
}
@@ -2292,7 +2292,7 @@ void WebContentsImpl::OnSaveURL(const GURL& url,
}
void WebContentsImpl::OnEnumerateDirectory(int request_id,
- const FilePath& path) {
+ const base::FilePath& path) {
if (!delegate_)
return;
@@ -2348,7 +2348,7 @@ void WebContentsImpl::OnOpenDateTimeDialog(int type, const std::string& value) {
#endif
-void WebContentsImpl::OnCrashedPlugin(const FilePath& plugin_path,
+void WebContentsImpl::OnCrashedPlugin(const base::FilePath& plugin_path,
base::ProcessId plugin_pid) {
FOR_EACH_OBSERVER(WebContentsObserver, observers_,
PluginCrashed(plugin_path, plugin_pid));
@@ -2381,7 +2381,7 @@ void WebContentsImpl::OnSetSelectedColorInColorChooser(int color_chooser_id,
}
void WebContentsImpl::OnPepperPluginHung(int plugin_child_id,
- const FilePath& path,
+ const base::FilePath& path,
bool is_hung) {
UMA_HISTOGRAM_COUNTS("Pepper.PluginHung", 1);
@@ -2401,7 +2401,7 @@ void WebContentsImpl::OnWebUISend(const GURL& source_url,
void WebContentsImpl::OnRequestPpapiBrokerPermission(
int request_id,
const GURL& url,
- const FilePath& plugin_path) {
+ const base::FilePath& plugin_path) {
if (!delegate_) {
OnPpapiBrokerPermissionResult(request_id, false);
return;
« no previous file with comments | « content/browser/web_contents/web_contents_impl.h ('k') | content/browser/web_contents/web_contents_view_aura.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698