Fix DevToolsProtocolTest.DoubleCrash with PlzNavigate on Android.

Fix DevToolsProtocolTest.DoubleCrash with PlzNavigate on Android.

This also fixes the test on Windows with and without PlzNavigate.

BUG=673806
Review-Url: https://codereview.chromium.org/2739493002
Cr-Commit-Position: refs/heads/master@{#455007}
Committed: https://chromium.googlesource.com/chromium/src/+/7e8bbb8e437db0b5cb65a6a84f1f3b0b67ca3a3e

[jam, 2017-03-06 21:01:14]

The CQ bit was checked by jam@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-06 21:01:40]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-06 21:37:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-06 21:37:12]

Dry run: This issue passed the CQ dry run.

[jam, 2017-03-06 22:41:25]

jam@chromium.org changed reviewers:
+ dgozman@chromium.org

[jam, 2017-03-06 22:41:26]

Seems like AgentHostClosed being called is normal when crashing the renderer,
i.e. with this callstack:

[5376:35408:0306/125821.157:1638456692:FATAL:devtools_protocol_browsertest.cc(393)]
Check failed: false.
Backtrace:
        content::DevToolsProtocolTest::AgentHostClosed
        content::DevToolsAgentHostImpl::ForceDetach
        content::RenderFrameDevToolsAgentHost::DestroyOnRenderFrameGone
        content::RenderFrameDevToolsAgentHost::FrameDeleted
        content::RenderFrameDevToolsAgentHost::RenderFrameDeleted
        content::WebContentsImpl::RenderFrameDeleted
        content::RenderFrameHostImpl::SetRenderFrameCreated
        content::RenderFrameHostImpl::OnRenderProcessGone
        base::DispatchToMethodImpl<content::RenderFrameHostImpl *,void
(__thiscall content::RenderFrameHostImpl::*)(int,int),std::tuple<int,
int> const &,0,1>
        base::DispatchToMethod<content::RenderFrameHostImpl *,void (__thiscall
content::RenderFrameHostImpl::*)(int,int),std::tuple<int,int>
 const &>
        IPC::DispatchToMethod<content::RenderFrameHostImpl,void (__thiscall
content::RenderFrameHostImpl::*)(int,int),void,std::tuple<int,in
t> >
       
IPC::MessageT<FrameHostMsg_RenderProcessGone_Meta,std::tuple<int,int>,void>::Dispatch<content::RenderFrameHostImpl,content::RenderFr
ameHostImpl,void,void (__thiscall content::RenderFrameHostImpl::*)(int,int)>
        content::RenderFrameHostImpl::OnMessageReceived
        content::RenderProcessHostImpl::ProcessDied
        content::RenderProcessHostImpl::OnChannelError

[dgozman, 2017-03-06 23:22:55]

lgtm

[jam, 2017-03-07 00:42:03]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2017-03-07 00:42:51]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-07 00:47:31]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488847323994860, "parent_rev":
"31f3265e09220f484c417341ca34acacf191600e", "commit_rev":
"7e8bbb8e437db0b5cb65a6a84f1f3b0b67ca3a3e"}

[commit-bot: I haz the power, 2017-03-07 00:48:36]

Description was changed from

==========
Fix DevToolsProtocolTest.DoubleCrash with PlzNavigate on Android.

This also fixes the test on Windows with and without PlzNavigate.

BUG=673806
==========

to

==========
Fix DevToolsProtocolTest.DoubleCrash with PlzNavigate on Android.

This also fixes the test on Windows with and without PlzNavigate.

BUG=673806

Review-Url: https://codereview.chromium.org/2739493002
Cr-Commit-Position: refs/heads/master@{#455007}
Committed:
https://chromium.googlesource.com/chromium/src/+/7e8bbb8e437db0b5cb65a6a84f1f...
==========

[commit-bot: I haz the power, 2017-03-07 00:48:37]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/7e8bbb8e437db0b5cb65a6a84f1f...