diff options
author | danakj@chromium.org <danakj@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-20 23:06:12 +0000 |
---|---|---|
committer | danakj@chromium.org <danakj@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-20 23:06:12 +0000 |
commit | e0e8421865e0d5356ab6ee2b3599850b53f7d8b1 (patch) | |
tree | 821406fda657c11bd333a57a4cf1f23967d95eca /cc/debug/rendering_stats.h | |
parent | def2970d531f10fb1c9d3953fdf67bc31444536f (diff) | |
download | chromium_src-e0e8421865e0d5356ab6ee2b3599850b53f7d8b1.zip chromium_src-e0e8421865e0d5356ab6ee2b3599850b53f7d8b1.tar.gz chromium_src-e0e8421865e0d5356ab6ee2b3599850b53f7d8b1.tar.bz2 |
Revert 189428 "cc: Chromify rendering_stats"
The naming change broke the scrolling benchmark or the perf waterfall.
http://build.chromium.org/p/chromium.perf/builders/Linux%2520Perf%2520%25281%2529/builds/21581/steps/scrolling_benchmark/logs/stdio
this enumerator is exposed API to JS extensions like the gpu benchmarking
extension, so this breaks benchmarking. We'll have to figure out how to
stage this with changes to the gpu benchmarking harness.
> cc: Chromify rendering_stats
>
>
> BUG=
>
> TBR=jamesr@chromium.org
>
> Review URL: https://chromiumcodereview.appspot.com/12780025
TBR=alexst@chromium.org
Review URL: https://codereview.chromium.org/12496017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@189448 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'cc/debug/rendering_stats.h')
-rw-r--r-- | cc/debug/rendering_stats.h | 81 |
1 files changed, 41 insertions, 40 deletions
diff --git a/cc/debug/rendering_stats.h b/cc/debug/rendering_stats.h index 33ea2c6..799bd88 100644 --- a/cc/debug/rendering_stats.h +++ b/cc/debug/rendering_stats.h @@ -12,51 +12,52 @@ namespace cc { struct CC_EXPORT RenderingStats { - int64 animation_frame_count; - int64 screen_frame_count; - 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. + // 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. - 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 |