[DevTools] Add notifyPopupOpeningObservers call before running debug loop

third_party/WebKit/Source/web/WebDevToolsAgentImpl.cpp

 /*
  * Copyright (C) 2010-2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 178 matching lines @@
         HeapVector<Member<WebFrameWidgetImpl>> widgets;
 
         // 1. Disable input events.
         const HashSet<WebViewImpl*>& viewImpls = WebViewImpl::allInstances();
         HashSet<WebViewImpl*>::const_iterator viewImplsEnd = viewImpls.end();
         for (HashSet<WebViewImpl*>::const_iterator it =  viewImpls.begin(); it != viewImplsEnd; ++it) {
             WebViewImpl* view = *it;
             m_frozenViews.add(view);
             views.append(view);
             view->setIgnoreInputEvents(true);
+            view->chromeClient().notifyPopupOpeningObservers();
         }
 
         const WebFrameWidgetsSet& widgetImpls = WebFrameWidgetImpl::allInstances();
         WebFrameWidgetsSet::const_iterator widgetImplsEnd = widgetImpls.end();
         for (WebFrameWidgetsSet::const_iterator it =  widgetImpls.begin(); it != widgetImplsEnd; ++it) {
             WebFrameWidgetImpl* widget = *it;
             m_frozenWidgets.add(widget);
             widgets.append(widget);
             widget->setIgnoreInputEvents(true);
+            widget->chromeClient().notifyPopupOpeningObservers();

[dgozman, 2016/05/26 19:27:29]

We don't need this.

[kozy, 2016/05/26 19:38:02]

Done.

         }
 
         // 2. Notify embedder about pausing.
         if (agent->client())
             agent->client()->willEnterDebugLoop();
 
         // 3. Disable active objects
         WebView::willEnterModalLoop();
 
         // 4. Process messages until quitNow is called.
@@ skipping 446 matching lines @@
 bool WebDevToolsAgent::shouldInterruptForMethod(const WebString& method)
 {
     return method == "Debugger.pause"
         || method == "Debugger.setBreakpoint"
         || method == "Debugger.setBreakpointByUrl"
         || method == "Debugger.removeBreakpoint"
         || method == "Debugger.setBreakpointsActive";
 }
 
 } // namespace blink