Propagate source_frame_number from LTH to DevTools

content/renderer/render_widget.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/render_widget.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
@@ skipping 1449 matching lines @@
   // Suppress updating when we are hidden.
   if (is_hidden_ || size_.IsEmpty() || is_swapped_out_) {
     paint_aggregator_.ClearPendingUpdate();
     needs_repainting_on_restore_ = true;
     TRACE_EVENT0("renderer", "EarlyOut_NotVisible");
     return;
   }
 
   // Tracking of frame rate jitter
   base::TimeTicks frame_begin_ticks = gfx::FrameTime::Now();
-  InstrumentWillBeginFrame();
+  InstrumentWillBeginFrame(0);

[jamesr, 2013/11/06 19:04:03]

why zero? we could be in compositing mode at this point

[caseq, 2013/11/07 06:01:45]

We only care about frame ids in the threaded mode, as they're only used to
associate main-thread frames with impl-side frames (and then potentially with
raster/image decode tasks). I can do that if you insist, but that would require
either exposing frame ids from LTH and generating them in another place within
LTH (we only tick source_frame_numbers on impl thread now), or generating them
in RenderWidget (this would also take care of non-composited case), so I'd
rather avoid doing this until we actually need it.

   AnimateIfNeeded();
 
   // Layout may generate more invalidation.  It may also enable the
   // GPU acceleration, so make sure to run layout before we send the
   // GpuRenderingActivated message.
   webwidget_->layout();
 
   // Check for whether we need to track swap buffers. We need to do that after
   // layout() because it may have switched us to accelerated compositing.
   if (is_accelerated_compositing_active_)
@@ skipping 1354 matching lines @@
           GetURLForGraphicsContext3D(),
           gpu_channel_host.get(),
           swap_client,
           attributes,
           false /* bind generates resources */,
           limits));
   return context.Pass();
 }
 
 }  // namespace content