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

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

Issue 1672983002: Fix download danger prompt buttons/actions. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix for tests Created 4 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.
(...skipping 308 matching lines...) Expand 10 before | Expand all | Expand 10 after
319 // Dangerous mode buttons. 319 // Dangerous mode buttons.
320 views::LabelButton* save_button_; 320 views::LabelButton* save_button_;
321 views::LabelButton* discard_button_; 321 views::LabelButton* discard_button_;
322 322
323 // Dangerous mode label. 323 // Dangerous mode label.
324 views::Label* dangerous_download_label_; 324 views::Label* dangerous_download_label_;
325 325
326 // Whether the dangerous mode label has been sized yet. 326 // Whether the dangerous mode label has been sized yet.
327 bool dangerous_download_label_sized_; 327 bool dangerous_download_label_sized_;
328 328
329 // The size of the buttons. Cached so animation works when hidden.
330 mutable gfx::Size cached_button_size_;
331
332 // Whether we are currently disabled as part of opening the downloaded file. 329 // Whether we are currently disabled as part of opening the downloaded file.
333 bool disabled_while_opening_; 330 bool disabled_while_opening_;
334 331
335 // The time at which this view was created. 332 // The time at which this view was created.
336 base::Time creation_time_; 333 base::Time creation_time_;
337 334
338 // The time at which a dangerous download warning was displayed. 335 // The time at which a dangerous download warning was displayed.
339 base::Time time_download_warning_shown_; 336 base::Time time_download_warning_shown_;
340 337
341 // The currently running download context menu. 338 // The currently running download context menu.
(...skipping 12 matching lines...) Expand all
354 scoped_ptr<extensions::ExperienceSamplingEvent> sampling_event_; 351 scoped_ptr<extensions::ExperienceSamplingEvent> sampling_event_;
355 352
356 // Method factory used to delay reenabling of the item when opening the 353 // Method factory used to delay reenabling of the item when opening the
357 // downloaded file. 354 // downloaded file.
358 base::WeakPtrFactory<DownloadItemView> weak_ptr_factory_; 355 base::WeakPtrFactory<DownloadItemView> weak_ptr_factory_;
359 356
360 DISALLOW_COPY_AND_ASSIGN(DownloadItemView); 357 DISALLOW_COPY_AND_ASSIGN(DownloadItemView);
361 }; 358 };
362 359
363 #endif // CHROME_BROWSER_UI_VIEWS_DOWNLOAD_DOWNLOAD_ITEM_VIEW_H__ 360 #endif // CHROME_BROWSER_UI_VIEWS_DOWNLOAD_DOWNLOAD_ITEM_VIEW_H__
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698