• Xida Chen's avatar
    [Code health] Unify two tracker type name array · 78ba8b7b
    Xida Chen authored
    We have kFrameSequenceTrackerTypeNames defined in the
    compositor_frame_reporter.cc and kBuiltinSequences defined in the
    frame_sequence_tracker.cc, where these two essential represents the same
    thing. This is problematic, that when we add an entry in one place, the
    other place must be adding that entry too.
    
    This CL defines kFrameSequenceTrackerTypeNames as static in the
    CompositorFrameReporter class and then FrameSequenceTracker can just
    use it.
    
    No behavior change should be introduced.
    
    Bug: None
    Change-Id: I0dc9e40be82d147615e1aa61afb1a7d8750390e8
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1774831
    Commit-Queue: Xida Chen <xidachen@chromium.org>
    Reviewed-by: default avatarSadrul Chowdhury <sadrul@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#692053}
    78ba8b7b
compositor_frame_reporter.cc 10.9 KB