• Fabrice de Gans-Riberi's avatar
    [fuchsia] Add support for remote_debugging_port in CreateContextParams. · 347f6990
    Fabrice de Gans-Riberi authored
    * Implement Frame.GetRemoteDebuggingPort.
    * Move all remote debugging handling in the Context process to
      WebEngineRemoteDebugging.
    * Change the debug DevTools behavior to not fallback on attempting to
      listen on IPv6 localhost in case of failure on IPv4 localhost.
    * Refactor the GetDevToolsListFromPort() test utility out of
      WebEngineDebugIntegrationTest to be shared with
      WebEngineIntegrationTest.
    * Fix a potential issue when the DevTools service fails to start and
      the debug DevTools listeners were not notified. In ContextImpl, the
      OnDevToolsPortReady() callback was only called when the DevTools
      service opened. It is now systematically called during browser
      process start-up and |devtools_port_received_| keeps track of that
      call in ContextImpl, rather than assuming a 0 value for
      |devtools_port_| means the service start was never requested.
    
    Bug: 973268
    Change-Id: I58d15c0656ebe1adfe6c254e4aefc0c7bfd54176
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1670441Reviewed-by: default avatarDirk Pranke <dpranke@chromium.org>
    Reviewed-by: default avatarSergey Ulanov <sergeyu@chromium.org>
    Commit-Queue: Fabrice de Gans-Riberi <fdegans@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#676208}
    347f6990
web_engine_browser_main_parts.cc 3.45 KB