• Sunny Sachanandani's avatar
    cc: Change scheduler state machine funnel names. · b0d272a7
    Sunny Sachanandani authored
    Changes the confusing ACTION_funnel_ names in scheduler state machine to
    did_ACTION_. Replaces prepare_tiles_funnel_ (the only real "funnel")
    with a bool that does the same job. Changes the state machine to use
    default member initialization.
    
    One SchedulerTest which explicitly tested that we reset the funnel after
    visibility change had to be modified. Since we don't reset the prepare
    tiles state on begin frame any more that old test doesn't work as is.
    Following along the discussion in crbug.com/497473#c14 the test's exact
    behavior isn't needed as we never end up in a situation with an ever
    increasing prepare tiles funnel after this change.
    
    R=danakj
    BUG=NONE
    
    CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
    
    Change-Id: I1541b358d05ad1b824955ae63e3bcb173c3ffda3
    Reviewed-on: https://chromium-review.googlesource.com/487322
    Commit-Queue: danakj <danakj@chromium.org>
    Reviewed-by: default avatardanakj <danakj@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#467994}
    b0d272a7
scheduler_state_machine.cc 47.3 KB