diff options
| author | mithro <mithro@mithis.com> | 2014-12-01 18:19:15 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2014-12-02 02:19:53 +0000 |
| commit | 06d1f3bfeece8b32c67bc41d4b4d7d6dcec01b32 (patch) | |
| tree | af9e0d08b38b6fe67a94e053719d50abdea8c1a9 /ui/compositor | |
| parent | 35a0b6ecedfd37e34aef8f927ad49a8b9e99cf5a (diff) | |
| download | chromium_src-06d1f3bfeece8b32c67bc41d4b4d7d6dcec01b32.zip chromium_src-06d1f3bfeece8b32c67bc41d4b4d7d6dcec01b32.tar.gz chromium_src-06d1f3bfeece8b32c67bc41d4b4d7d6dcec01b32.tar.bz2 | |
cc: Adding creation location to debug BeginFrameArgs objects.
This allows easy tracing of a BeginFrameArgs object back to the
BeginFrameSource that created it. This is very useful as we have multiple
BeginFrameSources in the system now. The Primary / Background sources being a
perfect example.
While location tracking is highly optimised (as it is used in every base::Bind
calls) it doubles the size of BeginFrameArgs objects. Hence we only enable
location tracking in debug builds. In release builds we make sure to never
create the Location objects in the first place, so no extra strings are found
in the binary. This can be checked with the strings tool.
BUG=346230
DEPS=742683002
Review URL: https://codereview.chromium.org/735723005
Cr-Commit-Position: refs/heads/master@{#306326}
Diffstat (limited to 'ui/compositor')
| -rw-r--r-- | ui/compositor/compositor.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ui/compositor/compositor.cc b/ui/compositor/compositor.cc index bbf05bb..c59930a 100644 --- a/ui/compositor/compositor.cc +++ b/ui/compositor/compositor.cc @@ -239,7 +239,7 @@ void Compositor::Draw() { // TODO(nduca): Temporary while compositor calls // compositeImmediately() directly. cc::BeginFrameArgs args = cc::BeginFrameArgs::Create( - gfx::FrameTime::Now(), base::TimeTicks(), + BEGINFRAME_FROM_HERE, gfx::FrameTime::Now(), base::TimeTicks(), cc::BeginFrameArgs::DefaultInterval(), cc::BeginFrameArgs::SYNCHRONOUS); BeginMainFrame(args); host_->Composite(args.frame_time); |