Prefer ScopedNestableTaskAllower over manual save/restore

webkit/support/webkit_support.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 "webkit/support/webkit_support.h"
 
 #include "base/at_exit.h"
 #include "base/base64.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 195 matching lines @@
 }
 
 class WebKitClientMessageLoopImpl
     : public WebDevToolsAgentClient::WebKitClientMessageLoop {
  public:
   WebKitClientMessageLoopImpl() : message_loop_(MessageLoop::current()) {}
   virtual ~WebKitClientMessageLoopImpl() {
     message_loop_ = NULL;
   }
   virtual void run() {
-    bool old_state = message_loop_->NestableTasksAllowed();
-    message_loop_->SetNestableTasksAllowed(true);
+    MessageLoop::ScopedNestableTaskAllower allow(message_loop_);
     message_loop_->Run();
-    message_loop_->SetNestableTasksAllowed(old_state);
   }
   virtual void quitNow() {
     message_loop_->QuitNow();
   }
  private:
   MessageLoop* message_loop_;
 };
 
 webkit_support::GraphicsContext3DImplementation
     g_graphics_context_3d_implementation =
@@ skipping 213 matching lines @@
 bool MessageLoopNestableTasksAllowed() {
   return MessageLoop::current()->NestableTasksAllowed();
 }
 
 void MessageLoopSetNestableTasksAllowed(bool allowed) {
   MessageLoop::current()->SetNestableTasksAllowed(allowed);
 }
 
 void DispatchMessageLoop() {
   MessageLoop* current = MessageLoop::current();
-  bool old_state = current->NestableTasksAllowed();
-  current->SetNestableTasksAllowed(true);
+  MessageLoop::ScopedNestableTaskAllower allow(current);
   current->RunAllPending();
-  current->SetNestableTasksAllowed(old_state);
 }
 
 WebDevToolsAgentClient::WebKitClientMessageLoop* CreateDevToolsMessageLoop() {
   return new WebKitClientMessageLoopImpl();
 }
 
 void PostDelayedTask(void (*func)(void*), void* context, int64 delay_ms) {
   MessageLoop::current()->PostDelayedTask(
       FROM_HERE,
       base::Bind(func, context),
@@ skipping 216 matching lines @@
 // Logging
 void EnableWebCoreLogChannels(const std::string& channels) {
   webkit_glue::EnableWebCoreLogChannels(channels);
 }
 
 void SetGamepadData(const WebKit::WebGamepads& pads) {
   test_environment->webkit_platform_support()->setGamepadData(pads);
 }
 
 }  // namespace webkit_support