Stop using nested message loop for alert() and other JS dialogs

content/renderer/render_frame_impl.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_RENDERER_RENDER_FRAME_IMPL_H_
 #define CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 832 matching lines @@
   // finally get right encoding of page.
   void UpdateEncoding(blink::WebFrame* frame,
                       const std::string& encoding_name);
 
   // Dispatches the current state of selection on the webpage to the browser if
   // it has changed.
   // TODO(varunjain): delete this method once we figure out how to keep
   // selection handles in sync with the webpage.
   void SyncSelectionIfRequired();
 
-  // Sends a message and runs a nested message loop.
-  bool SendAndRunNestedMessageLoop(IPC::SyncMessage* message);
+  // Sends a synchronous message.
+  bool SendSyncMessage(IPC::SyncMessage* message);

[aelias_OOO_until_Jul13, 2016/05/17 19:35:29]

This declaration has no implementation nor callers, please delete it.

[Changwan Ryu, 2016/05/19 00:44:56]

Done.

 
   bool RunJavaScriptMessage(JavaScriptMessageType type,
                             const base::string16& message,
                             const base::string16& default_value,
                             const GURL& frame_url,
                             base::string16* result);
 
   // Loads the appropriate error page for the specified failure into the frame.
   void LoadNavigationErrorPage(const blink::WebURLRequest& failed_request,
                                const blink::WebURLError& error,
@@ skipping 340 matching lines @@
   FrameBlameContext* blame_context_;  // Not owned.
 
   base::WeakPtrFactory<RenderFrameImpl> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_