Eliminate NOTIFICATION_HISTORY_LOADED notification

chrome/test/base/ui_test_utils.h

Index: chrome/test/base/ui_test_utils.h
diff --git a/chrome/test/base/ui_test_utils.h b/chrome/test/base/ui_test_utils.h
index 9e690d1534ad604a9efc066ee396884e928eba93..57a1d7019246a5bfc4af212fae87581a999a556c 100644
--- a/chrome/test/base/ui_test_utils.h
+++ b/chrome/test/base/ui_test_utils.h
@@ -306,6 +306,19 @@ class HistoryEnumerator {
   DISALLOW_COPY_AND_ASSIGN(HistoryEnumerator);
 };
 
+// Wait for HistoryService to load.

[sdefresne, 2014/09/23 08:45:10]

You can move this class in ui_test_utils.cc since it is not used from any other
file.

[nshaik, 2014/10/29 08:43:39]

Done.

+class WaitHistoryLoadedObserver : public history::HistoryServiceObserver {
+ public:
+  explicit WaitHistoryLoadedObserver(content::MessageLoopRunner* runner);
+  // Is called when |service| is loaded.

[sdefresne, 2014/10/20 13:15:43]

// history::HistoryServiceObserver:

+  virtual void HistoryServiceLoaded(HistoryService* service) OVERRIDE;

[sdefresne, 2014/10/20 13:15:43]

OVERRIDE -> override

+  virtual ~WaitHistoryLoadedObserver();

[sdefresne, 2014/10/20 13:15:43]

Move the destructor with the constructor.

+
+ private:
+  // weak
+  content::MessageLoopRunner* runner_;
+};
+
 }  // namespace ui_test_utils
 
 #endif  // CHROME_TEST_BASE_UI_TEST_UTILS_H_