• Hajime Hoshi's avatar
    Pass a task runner to base::Timer at RenderViewImpl · f3437441
    Hajime Hoshi authored
    This is part of efforts to replace base::ThreadTaskRunnerHandle::Get()
    and SequencedTaskRunnerHandle::Get() with other appropriate task runners
    in the renderer.
    
    This avoids the SequencedTaskRunnerHandle::Get() call when the constructor's
    default parameter is used. We are trying to ban SequencedTaskRunnerHandle::Get()
    in content/renderer because it usually is not the right task runner to use,
    as a per-frame runner should be used instead. In this case, it doesn't
    really matter since frames are not involved, but we still want to remove the
    calls for the bigger benefit of banning it in content/renderer.
    
    Bug: 786332
    Change-Id: I96a5c6657c029bd685e2302a778316b69b3af683
    Reviewed-on: https://chromium-review.googlesource.com/942287
    Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org>
    Reviewed-by: default avatarAlexander Timin <altimin@chromium.org>
    Reviewed-by: default avatarKentaro Hara <haraken@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#540820}
    f3437441
render_view_impl.h 33.4 KB