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

Side by Side Diff: chrome/browser/automation/automation_provider_observers.h

Issue 10805020: Kill DownloadItem::IsOtr() (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: . Created 8 years, 4 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 | 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 #ifndef CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_OBSERVERS_H_ 5 #ifndef CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_OBSERVERS_H_
6 #define CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_OBSERVERS_H_ 6 #define CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_OBSERVERS_H_
7 7
8 #include <deque> 8 #include <deque>
9 #include <map> 9 #include <map>
10 #include <set> 10 #include <set>
(...skipping 1074 matching lines...) Expand 10 before | Expand all | Expand 10 after
1085 }; 1085 };
1086 1086
1087 // Allows the automation provider to wait until the download has been updated 1087 // Allows the automation provider to wait until the download has been updated
1088 // or opened. 1088 // or opened.
1089 class AutomationProviderDownloadUpdatedObserver 1089 class AutomationProviderDownloadUpdatedObserver
1090 : public content::DownloadItem::Observer { 1090 : public content::DownloadItem::Observer {
1091 public: 1091 public:
1092 AutomationProviderDownloadUpdatedObserver( 1092 AutomationProviderDownloadUpdatedObserver(
1093 AutomationProvider* provider, 1093 AutomationProvider* provider,
1094 IPC::Message* reply_message, 1094 IPC::Message* reply_message,
1095 bool wait_for_open); 1095 bool wait_for_open,
1096 Profile* profile);
Randy Smith (Not in Mondays) 2012/08/02 17:52:17 It looks like there's an observer created per down
benjhayden 2012/08/13 15:08:09 Done.
1096 virtual ~AutomationProviderDownloadUpdatedObserver(); 1097 virtual ~AutomationProviderDownloadUpdatedObserver();
1097 1098
1098 virtual void OnDownloadUpdated(content::DownloadItem* download); 1099 virtual void OnDownloadUpdated(content::DownloadItem* download);
1099 virtual void OnDownloadOpened(content::DownloadItem* download); 1100 virtual void OnDownloadOpened(content::DownloadItem* download);
1100 1101
1101 private: 1102 private:
1102 base::WeakPtr<AutomationProvider> provider_; 1103 base::WeakPtr<AutomationProvider> provider_;
1103 scoped_ptr<IPC::Message> reply_message_; 1104 scoped_ptr<IPC::Message> reply_message_;
1104 bool wait_for_open_; 1105 bool wait_for_open_;
1106 Profile* profile_;
1105 1107
1106 DISALLOW_COPY_AND_ASSIGN(AutomationProviderDownloadUpdatedObserver); 1108 DISALLOW_COPY_AND_ASSIGN(AutomationProviderDownloadUpdatedObserver);
1107 }; 1109 };
1108 1110
1109 // Allows the automation provider to wait until the download model has changed 1111 // Allows the automation provider to wait until the download model has changed
1110 // (because a new download has been added or removed). 1112 // (because a new download has been added or removed).
1111 class AutomationProviderDownloadModelChangedObserver 1113 class AutomationProviderDownloadModelChangedObserver
1112 : public content::DownloadManager::Observer { 1114 : public content::DownloadManager::Observer {
1113 public: 1115 public:
1114 AutomationProviderDownloadModelChangedObserver( 1116 AutomationProviderDownloadModelChangedObserver(
(...skipping 872 matching lines...) Expand 10 before | Expand all | Expand 10 after
1987 base::WeakPtr<AutomationProvider> automation_; 1989 base::WeakPtr<AutomationProvider> automation_;
1988 scoped_ptr<IPC::Message> reply_message_; 1990 scoped_ptr<IPC::Message> reply_message_;
1989 int new_window_id_; 1991 int new_window_id_;
1990 int num_loads_; 1992 int num_loads_;
1991 1993
1992 DISALLOW_COPY_AND_ASSIGN( 1994 DISALLOW_COPY_AND_ASSIGN(
1993 BrowserOpenedWithExistingProfileNotificationObserver); 1995 BrowserOpenedWithExistingProfileNotificationObserver);
1994 }; 1996 };
1995 1997
1996 #endif // CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_OBSERVERS_H_ 1998 #endif // CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_OBSERVERS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698