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

Side by Side Diff: trunk/src/chrome/browser/download/download_request_infobar_delegate.cc

Issue 102163002: Revert 238283 "Infobar system refactor." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years 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) 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 #include "chrome/browser/download/download_request_infobar_delegate.h" 5 #include "chrome/browser/download/download_request_infobar_delegate.h"
6 6
7 #include "chrome/browser/infobars/infobar.h"
8 #include "chrome/browser/infobars/infobar_service.h" 7 #include "chrome/browser/infobars/infobar_service.h"
9 #include "grit/generated_resources.h" 8 #include "grit/generated_resources.h"
10 #include "grit/theme_resources.h" 9 #include "grit/theme_resources.h"
11 #include "ui/base/l10n/l10n_util.h" 10 #include "ui/base/l10n/l10n_util.h"
12 11
13 DownloadRequestInfoBarDelegate::FakeCreateCallback* 12 DownloadRequestInfoBarDelegate::FakeCreateCallback*
14 DownloadRequestInfoBarDelegate::callback_ = NULL; 13 DownloadRequestInfoBarDelegate::callback_ = NULL;
15 14
16 DownloadRequestInfoBarDelegate::~DownloadRequestInfoBarDelegate() { 15 DownloadRequestInfoBarDelegate::~DownloadRequestInfoBarDelegate() {
17 if (!responded_ && host_) 16 if (!responded_ && host_)
(...skipping 11 matching lines...) Expand all
29 // |web_contents| may not have a InfoBarService if it's actually a 28 // |web_contents| may not have a InfoBarService if it's actually a
30 // WebContents like those used for extension popups/bubbles and hosted apps 29 // WebContents like those used for extension popups/bubbles and hosted apps
31 // etc. 30 // etc.
32 // TODO(benjhayden): If this is an automatic download from an extension, 31 // TODO(benjhayden): If this is an automatic download from an extension,
33 // it would be convenient for the extension author if we send a message to 32 // it would be convenient for the extension author if we send a message to
34 // the extension's DevTools console (as we do for CSP) about how 33 // the extension's DevTools console (as we do for CSP) about how
35 // extensions should use chrome.downloads.download() (requires the 34 // extensions should use chrome.downloads.download() (requires the
36 // "downloads" permission) to automatically download >1 files. 35 // "downloads" permission) to automatically download >1 files.
37 host->Cancel(); 36 host->Cancel();
38 } else { 37 } else {
39 infobar_service->AddInfoBar(ConfirmInfoBarDelegate::CreateInfoBar( 38 infobar_service->AddInfoBar(scoped_ptr<InfoBarDelegate>(
40 scoped_ptr<ConfirmInfoBarDelegate>( 39 new DownloadRequestInfoBarDelegate(infobar_service, host)));
41 new DownloadRequestInfoBarDelegate(host))));
42 } 40 }
43 } 41 }
44 42
45 // static 43 // static
46 void DownloadRequestInfoBarDelegate::SetCallbackForTesting( 44 void DownloadRequestInfoBarDelegate::SetCallbackForTesting(
47 FakeCreateCallback* callback) { 45 FakeCreateCallback* callback) {
48 DownloadRequestInfoBarDelegate::callback_ = callback; 46 DownloadRequestInfoBarDelegate::callback_ = callback;
49 } 47 }
50 48
51 DownloadRequestInfoBarDelegate::DownloadRequestInfoBarDelegate( 49 DownloadRequestInfoBarDelegate::DownloadRequestInfoBarDelegate(
50 InfoBarService* infobar_service,
52 base::WeakPtr<DownloadRequestLimiter::TabDownloadState> host) 51 base::WeakPtr<DownloadRequestLimiter::TabDownloadState> host)
53 : ConfirmInfoBarDelegate(), 52 : ConfirmInfoBarDelegate(infobar_service),
54 responded_(false), 53 responded_(false),
55 host_(host) { 54 host_(host) {
56 } 55 }
57 56
58 int DownloadRequestInfoBarDelegate::GetIconID() const { 57 int DownloadRequestInfoBarDelegate::GetIconID() const {
59 return IDR_INFOBAR_MULTIPLE_DOWNLOADS; 58 return IDR_INFOBAR_MULTIPLE_DOWNLOADS;
60 } 59 }
61 60
62 string16 DownloadRequestInfoBarDelegate::GetMessageText() const { 61 string16 DownloadRequestInfoBarDelegate::GetMessageText() const {
63 return l10n_util::GetStringUTF16(IDS_MULTI_DOWNLOAD_WARNING); 62 return l10n_util::GetStringUTF16(IDS_MULTI_DOWNLOAD_WARNING);
(...skipping 17 matching lines...) Expand all
81 80
82 bool DownloadRequestInfoBarDelegate::Cancel() { 81 bool DownloadRequestInfoBarDelegate::Cancel() {
83 DCHECK(!responded_); 82 DCHECK(!responded_);
84 responded_ = true; 83 responded_ = true;
85 if (host_) { 84 if (host_) {
86 // This may invalidate |host_|. 85 // This may invalidate |host_|.
87 host_->Cancel(); 86 host_->Cancel();
88 } 87 }
89 return !host_; 88 return !host_;
90 } 89 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698