• Alex Moshchuk's avatar
    Fix the spellcheck enabled setting for remote-to-local main frame navigations. · 3314ae7b
    Alex Moshchuk authored
    Previously, the Blink spellchecking enabled state was initialized from
    ChromeContentRendererClient::RenderViewCreated(), which only did
    this for RenderViews where the main frame was a local frame.
    
    This CL moves that initialization logic into RenderFrameCreated(),
    where it is applied to all main frames.  Other than being good cleanup
    because RenderView is going away, this also makes it also work for
    provisional main frames (i.e., when the main frame is remote and it
    navigates back to a local frame).  The latter became more common
    after r501081, which means that the spellcheck enabled setting wasn't
    always respected after cross-process main frame navigations prior to
    this fix.
    
    Bug: 784673
    Change-Id: I91851c35027715b3d0c3a6ddfc22c514806adff3
    Reviewed-on: https://chromium-review.googlesource.com/794790Reviewed-by: default avatarXiaocheng Hu <xiaochengh@chromium.org>
    Reviewed-by: default avatarLei Zhang <thestig@chromium.org>
    Commit-Queue: Alex Moshchuk <alexmos@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#519917}
    3314ae7b
chrome_content_renderer_client.cc 67.5 KB