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

Side by Side Diff: chrome/browser/ui/views/download/download_item_view.h

Issue 153353006: Opt out of download feedback from settings (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Some cleanup Created 6 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 // A ChromeView that implements one download on the Download shelf. 5 // A ChromeView that implements one download on the Download shelf.
6 // Each DownloadItemView contains an application icon, a text label 6 // Each DownloadItemView contains an application icon, a text label
7 // indicating the download's file name, a text label indicating the 7 // indicating the download's file name, a text label indicating the
8 // download's status (such as the number of bytes downloaded so far) 8 // download's status (such as the number of bytes downloaded so far)
9 // and a button for canceling an in progress download, or opening 9 // and a button for canceling an in progress download, or opening
10 // the completed download. 10 // the completed download.
11 // 11 //
12 // The DownloadItemView lives in the Browser, and has a corresponding 12 // The DownloadItemView lives in the Browser, and has a corresponding
13 // DownloadController that receives / writes data which lives in the 13 // DownloadController that receives / writes data which lives in the
14 // Renderer. 14 // Renderer.
15 15
16 #ifndef CHROME_BROWSER_UI_VIEWS_DOWNLOAD_DOWNLOAD_ITEM_VIEW_H__ 16 #ifndef CHROME_BROWSER_UI_VIEWS_DOWNLOAD_DOWNLOAD_ITEM_VIEW_H__
17 #define CHROME_BROWSER_UI_VIEWS_DOWNLOAD_DOWNLOAD_ITEM_VIEW_H__ 17 #define CHROME_BROWSER_UI_VIEWS_DOWNLOAD_DOWNLOAD_ITEM_VIEW_H__
18 18
19 #include <string> 19 #include <string>
20 20
21 #include "base/basictypes.h" 21 #include "base/basictypes.h"
22 #include "base/memory/scoped_ptr.h" 22 #include "base/memory/scoped_ptr.h"
23 #include "base/memory/weak_ptr.h" 23 #include "base/memory/weak_ptr.h"
24 #include "base/strings/string_util.h" 24 #include "base/strings/string_util.h"
25 #include "base/task/cancelable_task_tracker.h" 25 #include "base/task/cancelable_task_tracker.h"
26 #include "base/time/time.h" 26 #include "base/time/time.h"
27 #include "base/timer/timer.h" 27 #include "base/timer/timer.h"
28 #include "chrome/browser/download/download_item_model.h" 28 #include "chrome/browser/download/download_item_model.h"
29 #include "chrome/browser/icon_manager.h" 29 #include "chrome/browser/icon_manager.h"
30 #include "chrome/browser/ui/views/download/download_feedback_dialog_view.h"
31 #include "content/public/browser/download_item.h" 30 #include "content/public/browser/download_item.h"
32 #include "content/public/browser/download_manager.h" 31 #include "content/public/browser/download_manager.h"
33 #include "ui/gfx/animation/animation_delegate.h" 32 #include "ui/gfx/animation/animation_delegate.h"
34 #include "ui/gfx/font_list.h" 33 #include "ui/gfx/font_list.h"
35 #include "ui/views/context_menu_controller.h" 34 #include "ui/views/context_menu_controller.h"
36 #include "ui/views/controls/button/button.h" 35 #include "ui/views/controls/button/button.h"
37 #include "ui/views/view.h" 36 #include "ui/views/view.h"
38 37
39 class DownloadShelfView; 38 class DownloadShelfView;
40 class DownloadShelfContextMenuView; 39 class DownloadShelfContextMenuView;
(...skipping 104 matching lines...) Expand 10 before | Expand all | Expand 10 after
145 gfx::ImageSkia* bottom; 144 gfx::ImageSkia* bottom;
146 }; 145 };
147 146
148 void OpenDownload(); 147 void OpenDownload();
149 148
150 // Submits the downloaded file to the safebrowsing download feedback service. 149 // Submits the downloaded file to the safebrowsing download feedback service.
151 // Returns whether submission was successful. On successful submission, 150 // Returns whether submission was successful. On successful submission,
152 // |this| and the DownloadItem will have been deleted. 151 // |this| and the DownloadItem will have been deleted.
153 bool SubmitDownloadToFeedbackService(); 152 bool SubmitDownloadToFeedbackService();
154 153
155 // If the user has enabled uploading, calls SubmitDownloadToFeedbackService. 154 // If the user has |enabled| uploading, calls SubmitDownloadToFeedbackService.
156 // Otherwise, it simply removes the DownloadItem without uploading. 155 // Otherwise, it simply removes the DownloadItem without uploading.
157 void PossiblySubmitDownloadToFeedbackService( 156 void PossiblySubmitDownloadToFeedbackService(bool enabled);
158 DownloadFeedbackDialogView::DownloadReportingStatus status);
159 157
160 void LoadIcon(); 158 void LoadIcon();
161 void LoadIconIfItemPathChanged(); 159 void LoadIconIfItemPathChanged();
162 160
163 // Update the button colors based on the current theme. 161 // Update the button colors based on the current theme.
164 void UpdateColorsFromTheme(); 162 void UpdateColorsFromTheme();
165 163
166 // Shows the context menu at the specified location. |point| is in the view's 164 // Shows the context menu at the specified location. |point| is in the view's
167 // coordinate system. 165 // coordinate system.
168 void ShowContextMenuImpl(const gfx::Point& point, 166 void ShowContextMenuImpl(const gfx::Point& point,
(...skipping 169 matching lines...) Expand 10 before | Expand all | Expand 10 after
338 336
339 // The icon loaded in the download shelf is based on the file path of the 337 // The icon loaded in the download shelf is based on the file path of the
340 // item. Store the path used, so that we can detect a change in the path 338 // item. Store the path used, so that we can detect a change in the path
341 // and reload the icon. 339 // and reload the icon.
342 base::FilePath last_download_item_path_; 340 base::FilePath last_download_item_path_;
343 341
344 DISALLOW_COPY_AND_ASSIGN(DownloadItemView); 342 DISALLOW_COPY_AND_ASSIGN(DownloadItemView);
345 }; 343 };
346 344
347 #endif // CHROME_BROWSER_UI_VIEWS_DOWNLOAD_DOWNLOAD_ITEM_VIEW_H__ 345 #endif // CHROME_BROWSER_UI_VIEWS_DOWNLOAD_DOWNLOAD_ITEM_VIEW_H__
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698