Fix for URL opening code

content/public/test/repeated_notification_observer.cc

+// Copyright (c) 2017 The Chromium Authors. All rights reserved.

[sky, 2017/06/08 19:17:49]

no (c)

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "content/public/test/repeated_notification_observer.h"
+
+#include "base/macros.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/browser/notification_service.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace content {
+
+RepeatedNotificationObserver::RepeatedNotificationObserver(int type, int count)
+    : num_outstanding_(count), running_(false) {
+  registrar_.Add(this, type, NotificationService::AllSources());
+}
+
+void RepeatedNotificationObserver::Observe(int type,
+                                           const NotificationSource& source,
+                                           const NotificationDetails& details) {
+  ASSERT_GT(num_outstanding_, 0);
+  if (!--num_outstanding_ && running_) {
+    BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,

[sky, 2017/06/08 19:17:49]

Can't you call run_loop_.Quit (or QuitWhenIdle()) directly?

+                            run_loop_.QuitClosure());
+  }
+}
+
+void RepeatedNotificationObserver::Wait() {
+  if (num_outstanding_ <= 0)
+    return;
+
+  running_ = true;

[sky, 2017/06/08 19:17:49]

base::AutoReset.

[eugenebng, 2017/06/09 13:16:22]

Done.

+  run_loop_.Run();
+  running_ = false;
+}
+
+}  // namespace content