Implement a policy to autologin a public account.

content/public/test/notification_watcher.h

+// Copyright 2013 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.
+
+#ifndef CONTENT_PUBLIC_TEST_NOTIFICATION_WATCHER_H_
+#define CONTENT_PUBLIC_TEST_NOTIFICATION_WATCHER_H_
+
+#include "base/basictypes.h"
+#include "base/callback.h"
+#include "base/compiler_specific.h"
+#include "base/run_loop.h"
+#include "content/public/browser/notification_observer.h"
+
+namespace content {
+
+class NotificationDetails;
+class NotificationSource;
+
+// Observes a specific notification type and quits the message loop once a
+// condition holds.
+class NotificationWatcher
+    : public content::NotificationObserver {
+ public:
+  // Callback invoked on notifications. Should return true when the condition
+  // that the caller is waiting for is satisfied.
+  typedef base::Callback<bool(void)> ConditionTestCallback;
+
+  explicit NotificationWatcher(int notification_type,
+                               const ConditionTestCallback& callback);
+  virtual ~NotificationWatcher();
+
+  void Run();

[sky, 2013/03/08 00:38:37]

This needs better documentation. I would never expect that if the callback
returns true it doesn't wait for the notification. In fact this class doesn't
seem particularly general and I would be inclined to keep it in your test. Are
there many users of it?

I also think the bool return value is ambiguous and an enum would be much
clearer.

+
+  // content::NotificationObserver:
+  virtual void Observe(int type,
+                       const content::NotificationSource& source,
+                       const content::NotificationDetails& details) OVERRIDE;
+
+ private:
+  int type_;

[sky, 2013/03/08 00:38:37]

const

+  ConditionTestCallback callback_;
+  base::RunLoop run_loop_;
+
+  DISALLOW_COPY_AND_ASSIGN(NotificationWatcher);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_PUBLIC_TEST_NOTIFICATION_WATCHER_H_