Chromium Code Reviews| Index: cc/debug/rendering_stats.h |
| diff --git a/cc/debug/rendering_stats.h b/cc/debug/rendering_stats.h |
| index 799bd887f34ac5440a9696aee689e54b97101bc6..7d6c18a52f0e242d84b268e16072a692cdc60ac7 100644 |
| --- a/cc/debug/rendering_stats.h |
| +++ b/cc/debug/rendering_stats.h |
| @@ -12,52 +12,53 @@ |
| namespace cc { |
| struct CC_EXPORT RenderingStats { |
| - // FIXME: Rename these to animationFrameCount and screenFrameCount, crbug.com/138641. |
| - int64 numAnimationFrames; |
| - int64 numFramesSentToScreen; |
| - int64 droppedFrameCount; |
| - base::TimeDelta totalPaintTime; |
| - base::TimeDelta totalRasterizeTime; |
| - base::TimeDelta totalRasterizeTimeForNowBinsOnPendingTree; |
| - base::TimeDelta totalCommitTime; |
| - int64 totalCommitCount; |
| - int64 totalPixelsPainted; |
| - int64 totalPixelsRasterized; |
| - int64 numImplThreadScrolls; |
| - int64 numMainThreadScrolls; |
| - int64 numLayersDrawn; |
| - int64 numMissingTiles; |
| - int64 totalDeferredImageDecodeCount; |
| - int64 totalDeferredImageCacheHitCount; |
| - int64 totalImageGatheringCount; |
| - base::TimeDelta totalDeferredImageDecodeTime; |
| - base::TimeDelta totalImageGatheringTime; |
| - // Note: when adding new members, please remember to update EnumerateFields |
| - // and Add in rendering_stats.cc. |
| + // FIXME: Rename these to animationFrameCount and screenFrameCount, |
|
danakj
2013/03/20 19:00:43
This bug is WontFix, remove this comment?
dtu
2013/03/20 19:08:08
Seems like it'd be easy enough to fix now. animati
|
| + // crbug.com/138641. |
| + int64 num_animation_frames; |
| + int64 num_frames_sent_to_screen; |
| + int64 dropped_frame_count; |
| + base::TimeDelta total_paint_time; |
| + base::TimeDelta total_rasterize_time; |
| + base::TimeDelta total_rasterize_time_for_now_bins_on_pending_tree; |
| + base::TimeDelta total_commit_time; |
| + int64 total_commit_count; |
| + int64 total_pixels_painted; |
| + int64 total_pixels_rasterized; |
| + int64 num_impl_thread_scrolls; |
| + int64 num_main_thread_scrolls; |
| + int64 num_layers_drawn; |
| + int64 num_missing_tiles; |
| + int64 total_deferred_image_decode_count; |
| + int64 total_deferred_image_cache_hit_count; |
| + int64 total_image_gathering_count; |
| + base::TimeDelta total_deferred_image_decode_time; |
| + base::TimeDelta total_image_gathering_time; |
| + // Note: when adding new members, please remember to update EnumerateFields |
| + // and Add in rendering_stats.cc. |
| - RenderingStats(); |
| + RenderingStats(); |
| - // In conjunction with enumerateFields, this allows the embedder to |
| - // enumerate the values in this structure without |
| - // having to embed references to its specific member variables. This |
| - // simplifies the addition of new fields to this type. |
| - class Enumerator { |
| - public: |
| - virtual void AddInt64(const char* name, int64 value) = 0; |
| - virtual void AddDouble(const char* name, double value) = 0; |
| - virtual void AddInt(const char* name, int value) = 0; |
| - virtual void AddTimeDeltaInSecondsF(const char* name, |
| - const base::TimeDelta& value) = 0; |
| + // In conjunction with enumerateFields, this allows the embedder to |
| + // enumerate the values in this structure without |
| + // having to embed references to its specific member variables. This |
| + // simplifies the addition of new fields to this type. |
| + class Enumerator { |
| + public: |
| + virtual void AddInt64(const char* name, int64 value) = 0; |
| + virtual void AddDouble(const char* name, double value) = 0; |
| + virtual void AddInt(const char* name, int value) = 0; |
| + virtual void AddTimeDeltaInSecondsF(const char* name, |
| + const base::TimeDelta& value) = 0; |
| - protected: |
| - virtual ~Enumerator() { } |
| - }; |
| + protected: |
| + virtual ~Enumerator() {} |
| + }; |
| - // Outputs the fields in this structure to the provided enumerator. |
| - void EnumerateFields(Enumerator* enumerator) const; |
| + // Outputs the fields in this structure to the provided enumerator. |
| + void EnumerateFields(Enumerator* enumerator) const; |
| - // Add fields of |other| to the fields in this structure. |
| - void Add(const RenderingStats& other); |
| + // Add fields of |other| to the fields in this structure. |
| + void Add(const RenderingStats& other); |
| }; |
| } // namespace cc |