• Charlie Harrison's avatar
    Revert "Predictor: Dispatch all predictions async from the resource throttle" · 35ae8708
    Charlie Harrison authored
    This reverts commit 24e730d2.
    
    Reason for revert: The delay navigation throttle will ruin data here.
    Let me re-land tomorrow.
    
    Original change's description:
    > Predictor: Dispatch all predictions async from the resource throttle
    > 
    > This is a small refactor that should have some performance wins. Now
    > that PlzNavigate has launched, we should be dispatching our preconnects
    > after the initial navigation request has gone out, not before. This is
    > because frequently (especially on Android), dispatching preconnects
    > tends to hog the IO thread, delaying the initial request.
    > 
    > Note that this predictor is probably going away, but this CPU work
    > is in the critical path of some other work I am planning on tackling.
    > 
    > Bug: 792524
    > Change-Id: I87e3c235be0480d766e3e45c9a97c4a26b15f60d
    > Reviewed-on: https://chromium-review.googlesource.com/821090
    > Commit-Queue: Charlie Harrison <csharrison@chromium.org>
    > Reviewed-by: Matt Menke <mmenke@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#523473}
    
    TBR=mmenke@chromium.org,csharrison@chromium.org
    
    Change-Id: I596b40d4d3f527de7e0df6c00f376281a2dbe381
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 792524
    Reviewed-on: https://chromium-review.googlesource.com/823110Reviewed-by: default avatarCharlie Harrison <csharrison@chromium.org>
    Commit-Queue: Charlie Harrison <csharrison@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#523486}
    35ae8708
predictor_tab_helper.cc 2.58 KB