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

Side by Side Diff: content/browser/tab_contents/tab_contents.cc

Issue 6973052: When the download folder does not exist, change the download folder to a user's "Downloads" (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Removed the change in url_request_mock_http_job.h Created 9 years, 4 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 "content/browser/tab_contents/tab_contents.h" 5 #include "content/browser/tab_contents/tab_contents.h"
6 6
7 #include <cmath> 7 #include <cmath>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/metrics/histogram.h" 10 #include "base/metrics/histogram.h"
11 #include "base/metrics/stats_counters.h" 11 #include "base/metrics/stats_counters.h"
12 #include "base/string16.h" 12 #include "base/string16.h"
13 #include "base/string_util.h" 13 #include "base/string_util.h"
14 #include "base/time.h" 14 #include "base/time.h"
15 #include "base/utf_string_conversions.h" 15 #include "base/utf_string_conversions.h"
16 #include "chrome/browser/download/download_manager.h" 16 #include "chrome/browser/download/download_manager.h"
17 #include "chrome/browser/download/download_request_limiter.h" 17 #include "chrome/browser/download/download_request_limiter.h"
18 #include "chrome/browser/download/download_util.h" 18 #include "chrome/browser/download/download_util.h"
19 #include "content/browser/browser_context.h" 19 #include "content/browser/browser_context.h"
20 #include "chrome/browser/download/save_package_file_picker.h"
20 #include "content/browser/child_process_security_policy.h" 21 #include "content/browser/child_process_security_policy.h"
21 #include "content/browser/content_browser_client.h" 22 #include "content/browser/content_browser_client.h"
22 #include "content/browser/debugger/devtools_manager.h" 23 #include "content/browser/debugger/devtools_manager.h"
23 #include "content/browser/host_zoom_map.h" 24 #include "content/browser/host_zoom_map.h"
24 #include "content/browser/in_process_webkit/session_storage_namespace.h" 25 #include "content/browser/in_process_webkit/session_storage_namespace.h"
25 #include "content/browser/load_from_memory_cache_details.h" 26 #include "content/browser/load_from_memory_cache_details.h"
26 #include "content/browser/load_notification_details.h" 27 #include "content/browser/load_notification_details.h"
27 #include "content/browser/renderer_host/render_process_host.h" 28 #include "content/browser/renderer_host/render_process_host.h"
28 #include "content/browser/renderer_host/render_view_host.h" 29 #include "content/browser/renderer_host/render_view_host.h"
29 #include "content/browser/renderer_host/render_widget_host_view.h" 30 #include "content/browser/renderer_host/render_widget_host_view.h"
(...skipping 723 matching lines...) Expand 10 before | Expand all | Expand 10 after
753 754
754 Stop(); 755 Stop();
755 756
756 // Create the save package and possibly prompt the user for the name to save 757 // Create the save package and possibly prompt the user for the name to save
757 // the page as. The user prompt is an asynchronous operation that runs on 758 // the page as. The user prompt is an asynchronous operation that runs on
758 // another thread. 759 // another thread.
759 save_package_ = new SavePackage(this); 760 save_package_ = new SavePackage(this);
760 save_package_->GetSaveInfo(); 761 save_package_->GetSaveInfo();
761 } 762 }
762 763
763 // Used in automated testing to bypass prompting the user for file names.
764 // Instead, the names and paths are hard coded rather than running them through
765 // file name sanitation and extension / mime checking.
766 bool TabContents::SavePage(const FilePath& main_file, const FilePath& dir_path, 764 bool TabContents::SavePage(const FilePath& main_file, const FilePath& dir_path,
767 SavePackage::SavePackageType save_type) { 765 SavePackage::SavePackageType save_type) {
768 // Stop the page from navigating. 766 // Stop the page from navigating.
769 Stop(); 767 Stop();
770 768
771 save_package_ = new SavePackage(this, save_type, main_file, dir_path); 769 save_package_ = new SavePackage(this, save_type, main_file, dir_path);
770 // Skips GetSaveInfo() and directly calls Init().
771 // We do not have to explicitly disable the select file dialog
772 // since we skip the process that can show the dialog.
772 return save_package_->Init(); 773 return save_package_->Init();
773 } 774 }
774 775
776 string16 TabContents::SavePageBasedOnDefaultPrefs() {
777 Stop();
778
779 save_package_ = new SavePackage(this);
780 // Disables the select file dialog.
781 SavePackageFilePicker::SetShouldPromptUser(false);
782 // This GetSaveInfo() calls save_package_->Init() in the background.
783 save_package_->GetSaveInfo();
784 return GetTitle();
785 }
786
775 void TabContents::OnSaveURL(const GURL& url) { 787 void TabContents::OnSaveURL(const GURL& url) {
776 DownloadManager* dlm = browser_context()->GetDownloadManager(); 788 DownloadManager* dlm = browser_context()->GetDownloadManager();
777 dlm->DownloadUrl(url, GetURL(), "", this); 789 dlm->DownloadUrl(url, GetURL(), "", this);
778 } 790 }
779 791
780 bool TabContents::IsActiveEntry(int32 page_id) { 792 bool TabContents::IsActiveEntry(int32 page_id) {
781 NavigationEntry* active_entry = controller_.GetActiveEntry(); 793 NavigationEntry* active_entry = controller_.GetActiveEntry();
782 return (active_entry != NULL && 794 return (active_entry != NULL &&
783 active_entry->site_instance() == GetSiteInstance() && 795 active_entry->site_instance() == GetSiteInstance() &&
784 active_entry->page_id() == page_id); 796 active_entry->page_id() == page_id);
(...skipping 1135 matching lines...) Expand 10 before | Expand all | Expand 10 after
1920 } 1932 }
1921 1933
1922 void TabContents::SwapInRenderViewHost(RenderViewHost* rvh) { 1934 void TabContents::SwapInRenderViewHost(RenderViewHost* rvh) {
1923 render_manager_.SwapInRenderViewHost(rvh); 1935 render_manager_.SwapInRenderViewHost(rvh);
1924 } 1936 }
1925 1937
1926 void TabContents::CreateViewAndSetSizeForRVH(RenderViewHost* rvh) { 1938 void TabContents::CreateViewAndSetSizeForRVH(RenderViewHost* rvh) {
1927 RenderWidgetHostView* rwh_view = view()->CreateViewForWidget(rvh); 1939 RenderWidgetHostView* rwh_view = view()->CreateViewForWidget(rvh);
1928 rwh_view->SetSize(view()->GetContainerSize()); 1940 rwh_view->SetSize(view()->GetContainerSize());
1929 } 1941 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698