• Lukasz Anforowicz's avatar
    Removing callers of the WebContents::GetRenderProcessHost() method. · bac91940
    Lukasz Anforowicz authored
    WebContents::GetRenderProcessHost is an API that doesn't behave as
    expected with out-of-process iframes, as multiple processes can be
    associated with a single WebContents.  This CL replaces callers of this
    API, so that they go through an equivalent API that explicitly selects
    which frame's process is needed.
    This CL was started by using an ad-hoc clang-plugin that replaced
    existing callsites of WebContents::GetRenderProcessHost() with a call to
    something like wc->GetMainFrame()->GetProcess().  This was followed-up
    by manually adding an include of render_frame_host.h and git cl format
    and git cl lint and then a self-review with small tweaks (e.g. sometimes
    using a wc->GetRenderViewHost()->GetProcess() is more appropriate).
    This CL was uploaded by git cl split.
    Bug: 666525
    Change-Id: I583ec4076732a84814ed05e2fe06b1d07ded6dee
    Reviewed-on: https://chromium-review.googlesource.com/688760Reviewed-by: default avatarJeffrey Yasskin <jyasskin@chromium.org>
    Reviewed-by: default avatarConley Owens <cco3@chromium.org>
    Commit-Queue: Jeffrey Yasskin <jyasskin@chromium.org>
    Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#506116}
web_bluetooth_browsertest.cc 7.44 KB