• kylechar's avatar
    Revert "Add debug BeginFrame tracking" · 5090beee
    kylechar authored
    This reverts commit 61bb69af.
    
    Reason for revert: Removing debug instrumentation before branch.
    
    Original change's description:
    > Add debug BeginFrame tracking
    > 
    > Add some temporary member variables to BeginFrameTracker that stores the
    > total sent/received and the last 10 sent/received BeginFrameIds. Despite
    > adding logic to stop sending OnBeginFrame() to unresponsive clients it
    > didn't help with the large number of unread messages in mojo message
    > queue. This information will do a better job of verifying if client is
    > in fact reading (and responding to) the OnBeginFrame() messages.
    > 
    > The |begin_frame_tracker_| is included in a few minidumps I looked at
    > it, so this information should be available in crash reports. Otherwise
    > we'll have to copy the information into the stack in OnBeginFrame().
    > 
    > Bug: 1011441
    > Change-Id: I44112d933fb4470fb037e7bfd2d9dc05dd826a94
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1994093
    > Reviewed-by: Saman Sami <samans@chromium.org>
    > Commit-Queue: kylechar <kylechar@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#730310}
    
    TBR=kylechar@chromium.org,samans@chromium.org
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Bug: 1011441
    Change-Id: I3c7d5572c19ede7d653d24175d7023e7a7e282c5
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2025768Reviewed-by: default avatarkylechar <kylechar@chromium.org>
    Commit-Queue: kylechar <kylechar@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#736090}
    5090beee
begin_frame_tracker.h 1.65 KB