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

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

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

Powered by Google App Engine
This is Rietveld 408576698