Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1016)

Unified Diff: cc/output/begin_frame_args.h

Issue 1533773002: Delete CC. (Closed) Base URL: git@github.com:domokit/mojo.git@cl-2e
Patch Set: rebase Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/debug/traced_value.cc ('k') | cc/output/begin_frame_args.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/output/begin_frame_args.h
diff --git a/cc/output/begin_frame_args.h b/cc/output/begin_frame_args.h
deleted file mode 100644
index 60788f7b1267a5ea8a614a03d4f8ce437c5dc76f..0000000000000000000000000000000000000000
--- a/cc/output/begin_frame_args.h
+++ /dev/null
@@ -1,96 +0,0 @@
-// Copyright 2013 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.
-
-#ifndef CC_OUTPUT_BEGIN_FRAME_ARGS_H_
-#define CC_OUTPUT_BEGIN_FRAME_ARGS_H_
-
-#include "base/location.h"
-#include "base/memory/ref_counted.h"
-#include "base/time/time.h"
-#include "base/values.h"
-
-namespace base {
-namespace trace_event {
-class ConvertableToTraceFormat;
-class TracedValue;
-}
-}
-
-/**
- * In debug builds we trace the creation origin of BeginFrameArgs objects. We
- * reuse the tracked_objects::Location system to do that.
- *
- * However, in release builds we don't want this as it doubles the size of the
- * BeginFrameArgs object. As well it adds a number of largish strings to the
- * binary. Despite the argument being unused, most compilers are unable to
- * optimise it away even when unused. Instead we use the BEGINFRAME_FROM_HERE
- * macro to prevent the data even getting referenced.
- */
-#ifdef NDEBUG
-#define BEGINFRAME_FROM_HERE nullptr
-#else
-#define BEGINFRAME_FROM_HERE FROM_HERE
-#endif
-
-namespace cc {
-
-struct BeginFrameArgs {
- enum BeginFrameArgsType {
- INVALID,
- NORMAL,
- SYNCHRONOUS,
- MISSED,
- // Not a real type, but used by the IPC system. Should always remain the
- // *last* value in this enum.
- BEGIN_FRAME_ARGS_TYPE_MAX,
- };
- static const char* TypeToString(BeginFrameArgsType type);
-
- // Creates an invalid set of values.
- BeginFrameArgs();
-
-#ifdef NDEBUG
- typedef const void* CreationLocation;
-#else
- typedef const tracked_objects::Location& CreationLocation;
- tracked_objects::Location created_from;
-#endif
-
- // You should be able to find all instances where a BeginFrame has been
- // created by searching for "BeginFrameArgs::Create".
- // The location argument should **always** be BEGINFRAME_FROM_HERE macro.
- static BeginFrameArgs Create(CreationLocation location,
- base::TimeTicks frame_time,
- base::TimeTicks deadline,
- base::TimeDelta interval,
- BeginFrameArgsType type);
-
- // This is the default delta that will be used to adjust the deadline when
- // proper draw-time estimations are not yet available.
- static base::TimeDelta DefaultEstimatedParentDrawTime();
-
- // This is the default interval to use to avoid sprinkling the code with
- // magic numbers.
- static base::TimeDelta DefaultInterval();
-
- bool IsValid() const { return interval >= base::TimeDelta(); }
-
- scoped_refptr<base::trace_event::ConvertableToTraceFormat> AsValue() const;
- void AsValueInto(base::trace_event::TracedValue* dict) const;
-
- base::TimeTicks frame_time;
- base::TimeTicks deadline;
- base::TimeDelta interval;
- BeginFrameArgsType type;
-
- private:
- BeginFrameArgs(base::TimeTicks frame_time,
- base::TimeTicks deadline,
- base::TimeDelta interval,
- BeginFrameArgsType type);
-};
-
-} // namespace cc
-
-#endif // CC_OUTPUT_BEGIN_FRAME_ARGS_H_
« no previous file with comments | « cc/debug/traced_value.cc ('k') | cc/output/begin_frame_args.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698