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

Side by Side Diff: chrome/browser/extensions/extension_infobar_delegate.cc

Issue 5794003: Deinline even more destructors. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 10 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) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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/extensions/extension_infobar_delegate.h" 5 #include "chrome/browser/extensions/extension_infobar_delegate.h"
6 6
7 #include "chrome/browser/extensions/extension_host.h" 7 #include "chrome/browser/extensions/extension_host.h"
8 #include "chrome/browser/extensions/extension_process_manager.h" 8 #include "chrome/browser/extensions/extension_process_manager.h"
9 #include "chrome/browser/profiles/profile.h" 9 #include "chrome/browser/profiles/profile.h"
10 #include "chrome/browser/tab_contents/tab_contents.h" 10 #include "chrome/browser/tab_contents/tab_contents.h"
(...skipping 20 matching lines...) Expand all
31 Source<Profile>(browser->profile())); 31 Source<Profile>(browser->profile()));
32 registrar_.Add(this, NotificationType::EXTENSION_UNLOADED, 32 registrar_.Add(this, NotificationType::EXTENSION_UNLOADED,
33 Source<Profile>(browser->profile())); 33 Source<Profile>(browser->profile()));
34 } 34 }
35 35
36 ExtensionInfoBarDelegate::~ExtensionInfoBarDelegate() { 36 ExtensionInfoBarDelegate::~ExtensionInfoBarDelegate() {
37 if (observer_) 37 if (observer_)
38 observer_->OnDelegateDeleted(); 38 observer_->OnDelegateDeleted();
39 } 39 }
40 40
41 void ExtensionInfoBarDelegate::InfoBarDismissed() {
42 closing_ = true;
43 }
44
41 bool ExtensionInfoBarDelegate::EqualsDelegate(InfoBarDelegate* delegate) const { 45 bool ExtensionInfoBarDelegate::EqualsDelegate(InfoBarDelegate* delegate) const {
42 ExtensionInfoBarDelegate* extension_delegate = 46 ExtensionInfoBarDelegate* extension_delegate =
43 delegate->AsExtensionInfoBarDelegate(); 47 delegate->AsExtensionInfoBarDelegate();
44 // When an extension crashes, an InfoBar is shown (for the crashed extension). 48 // When an extension crashes, an InfoBar is shown (for the crashed extension).
45 // That will result in a call to this function (to see if this InfoBarDelegate 49 // That will result in a call to this function (to see if this InfoBarDelegate
46 // is already showing the 'extension crashed InfoBar', which it never is), but 50 // is already showing the 'extension crashed InfoBar', which it never is), but
47 // if it is our extension that crashes, the extension delegate is NULL so 51 // if it is our extension that crashes, the extension delegate is NULL so
48 // we cannot check. 52 // we cannot check.
49 if (!extension_delegate) 53 if (!extension_delegate)
50 return false; 54 return false;
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
82 if (extension_ == extension) 86 if (extension_ == extension)
83 tab_contents_->RemoveInfoBar(this); 87 tab_contents_->RemoveInfoBar(this);
84 break; 88 break;
85 } 89 }
86 default: { 90 default: {
87 NOTREACHED() << "Unknown message"; 91 NOTREACHED() << "Unknown message";
88 break; 92 break;
89 } 93 }
90 } 94 }
91 } 95 }
OLDNEW
« no previous file with comments | « chrome/browser/extensions/extension_infobar_delegate.h ('k') | chrome/browser/extensions/extensions_ui.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698