• Majid Valipour's avatar
    [cc] Rewrite keyframe model timing computation based on local time · c5543eb5
    Majid Valipour authored
    Formalize the concept of local time and use it internally where we would
    have done a one-off calculations before.
    
    In this process I noticed and documented issues how pause time is being
    applied. In particular pause time incorrectly include offset_time_ and
    excludes total_paused_duration_. I suspect these issues flew under the
    radar mainly because pausing is only ever used for simple test cases not
    involving animations with start delays or multiple pause/unpause cycles.
    I plan to fix this issues in a follow up patch.
    
    
    Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
    Change-Id: I02030dbd5ef0656b2738353a6e468b81c9b259a8
    Reviewed-on: https://chromium-review.googlesource.com/1037635
    Commit-Queue: Majid Valipour <majidvp@chromium.org>
    Reviewed-by: default avatarRobert Flack <flackr@chromium.org>
    Reviewed-by: default avatarYi Gu <yigu@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574949}
    c5543eb5
keyframe_model.h 10.8 KB