content: remove NOTIFICATION_SAVE_PACKAGE_SUCCESSFULLY_FINISHED

content/public/test/test_utils.cc

 // Copyright (c) 2012 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.
 
 #include "content/public/test/test_utils.h"
 
 #include "base/bind.h"
 #include "base/message_loop.h"
 #include "base/run_loop.h"
 #include "base/utf_string_conversions.h"
@@ skipping 114 matching lines @@
 
   render_view_host->ExecuteJavascriptInWebFrameCallbackResult(
       string16(),  // frame_xpath,
       UTF8ToUTF16(script),
       base::Bind(&ScriptCallback::ResultCallback, base::Unretained(&observer)));
   MessageLoop* loop = MessageLoop::current();
   loop->Run();
   return observer.result().Pass();
 }
 
-MessageLoopRunner::MessageLoopRunner() {
+MessageLoopRunner::MessageLoopRunner()
+    : loop_running_(false),
+      quit_closure_called_(false) {
 }
 
 MessageLoopRunner::~MessageLoopRunner() {
 }
 
 void MessageLoopRunner::Run() {
+  // Do not run the message loop if our quit closure has already been called.
+  // This helps in scenarios where the closure has a chance to run before
+  // we Run explicitly.
+  if (quit_closure_called_)
+    return;
+
+  loop_running_ = true;
   RunThisRunLoop(&run_loop_);
 }
 
 base::Closure MessageLoopRunner::QuitClosure() {
   return base::Bind(&MessageLoopRunner::Quit, this);
 }
 
 void MessageLoopRunner::Quit() {
-  GetQuitTaskForRunLoop(&run_loop_).Run();
+  quit_closure_called_ = true;
+
+  // Only run the quit task if we are running the message loop.
+  if (loop_running_) {
+    GetQuitTaskForRunLoop(&run_loop_).Run();
+    loop_running_ = false;
+  }
 }
 
 WindowedNotificationObserver::WindowedNotificationObserver(
     int notification_type,
     const NotificationSource& source)
     : seen_(false),
       running_(false),
       source_(NotificationService::AllSources()) {
   registrar_.Add(this, notification_type, source);
 }
@@ skipping 18 matching lines @@
   details_ = details;
   seen_ = true;
   if (!running_)
     return;
 
   message_loop_runner_->Quit();
   running_ = false;
 }
 
 }  // namespace content