Clean up warnings found in our testing code.

Time wise, this only speeds things up by a minute, which I think is within the
margin of error. I can't get a good size measurement since we aren't building
.a files for a lot of this code.

BUG=none
TEST=compiles

Review URL: http://codereview.chromium.org/6312137

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73999 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 19 insertions, 1 deletions

diff --git a/chrome/test/ui_test_utils.cc b/chrome/test/ui_test_utils.cc
index 873bfe1..436273b 100644
--- a/chrome/test/ui_test_utils.cc
+++ b/chrome/test/ui_test_utils.cc
@@ -1,4 +1,4 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
@@ -873,6 +873,20 @@ TestWebSocketServer::~TestWebSocketServer() {
   base::LaunchApp(*cmd_line.get(), true, false, NULL);
 }
 
+TestNotificationObserver::TestNotificationObserver()
+    : source_(NotificationService::AllSources()) {
+}
+
+TestNotificationObserver::~TestNotificationObserver() {}
+
+void TestNotificationObserver::Observe(NotificationType type,
+                                       const NotificationSource& source,
+                                       const NotificationDetails& details) {
+  source_ = source;
+  details_ = details;
+  MessageLoopForUI::current()->Quit();
+}
+
 WindowedNotificationObserver::WindowedNotificationObserver(
     NotificationType notification_type,
     const NotificationSource& source)
@@ -882,6 +896,8 @@ WindowedNotificationObserver::WindowedNotificationObserver(
   registrar_.Add(this, notification_type, waiting_for_);
 }
 
+WindowedNotificationObserver::~WindowedNotificationObserver() {}
+
 void WindowedNotificationObserver::Wait() {
   if (waiting_for_ == NotificationService::AllSources()) {
     LOG(FATAL) << "Wait called when monitoring all sources. You must use "
@@ -926,6 +942,8 @@ DOMMessageQueue::DOMMessageQueue() {
                  NotificationService::AllSources());
 }
 
+DOMMessageQueue::~DOMMessageQueue() {}
+
 void DOMMessageQueue::Observe(NotificationType type,
                               const NotificationSource& source,
                               const NotificationDetails& details) {