GitLab

DevToolsWindowCreationObserver::WaitForLoad clears its MessageLoopRunner
right when DevToolsWindowCreated is called, this means the runner would
be destroyed while its QuitClosure is running. This CL fixes the issue
by moving the destruction after MessageLoopRunner::Run().

This bug is evident when there is no DevToolsWindow at DTWCObserver
instantiation time. This CL adds a regression test for that.

Bug: 1126105
Change-Id: I68b122c657abbc9a0ecca7875ed19dc6d33914eb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363886Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Reviewed-by: Istiaque Ahmed <lazyboy@chromium.org>
Commit-Queue: Ghazale Hosseinabadi <ghazale@chromium.org>
Cr-Commit-Position: refs/heads/master@{#805859}