Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 #pragma once | 7 #pragma once |
| 8 | 8 |
| 9 #include <deque> | 9 #include <deque> |
| 10 #include <map> | 10 #include <map> |
| (...skipping 1839 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 1850 | 1850 |
| 1851 private: | 1851 private: |
| 1852 base::WeakPtr<AutomationProvider> automation_; | 1852 base::WeakPtr<AutomationProvider> automation_; |
| 1853 scoped_ptr<IPC::Message> reply_message_; | 1853 scoped_ptr<IPC::Message> reply_message_; |
| 1854 std::string extension_id_; | 1854 std::string extension_id_; |
| 1855 content::NotificationRegistrar registrar_; | 1855 content::NotificationRegistrar registrar_; |
| 1856 | 1856 |
| 1857 DISALLOW_COPY_AND_ASSIGN(ExtensionPopupObserver); | 1857 DISALLOW_COPY_AND_ASSIGN(ExtensionPopupObserver); |
| 1858 }; | 1858 }; |
| 1859 | 1859 |
| 1860 // Allows the automation provider to wait for a WINDOW_MAXIMIZED notification. | |
| 1861 class WindowMaximizedObserver : public content::NotificationObserver { | |
| 1862 public: | |
| 1863 WindowMaximizedObserver(AutomationProvider* automation, | |
| 1864 IPC::Message* reply_message); | |
|
kkania
2012/05/30 01:03:58
when defining or prototyping a function, the args,
zori
2012/05/31 23:38:22
Done.
| |
| 1865 virtual ~WindowMaximizedObserver(); | |
| 1866 | |
| 1867 virtual void Observe(int type, | |
|
kkania
2012/05/30 01:03:58
It's typical style to put:
// Overridden from Noti
zori
2012/05/31 23:38:22
Done.
| |
| 1868 const content::NotificationSource& source, | |
| 1869 const content::NotificationDetails& details); | |
|
kkania
2012/05/30 01:03:58
add OVERRIDE macro
zori
2012/05/31 23:38:22
Done.
| |
| 1870 | |
| 1871 private: | |
| 1872 content::NotificationRegistrar registrar_; | |
| 1873 base::WeakPtr<AutomationProvider> automation_; | |
| 1874 scoped_ptr<IPC::Message> reply_message_; | |
| 1875 | |
| 1876 DISALLOW_COPY_AND_ASSIGN(WindowMaximizedObserver); | |
| 1877 }; | |
| 1878 | |
| 1879 | |
| 1880 | |
| 1860 #endif // CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_OBSERVERS_H_ | 1881 #endif // CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_OBSERVERS_H_ |
| OLD | NEW |