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

Unified Diff: cc/output/latency_info_swap_promise.cc

Issue 1126963006: Move VISUAL_STATE promise to activation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Current state (presubmit warnings, cc_unittests tests failing) Created 5 years, 7 months 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
Index: cc/output/latency_info_swap_promise.cc
diff --git a/cc/output/latency_info_swap_promise.cc b/cc/output/latency_info_swap_promise.cc
index 3c0716294adf3a99e1cf3fab4ba0350d42aea931..7a92d3295b9bfe3a9e839947c5f9cebf24a1da27 100644
--- a/cc/output/latency_info_swap_promise.cc
+++ b/cc/output/latency_info_swap_promise.cc
@@ -8,14 +8,14 @@
namespace {
ui::LatencyComponentType DidNotSwapReasonToLatencyComponentType(
- cc::SwapPromise::DidNotSwapReason reason) {
+ cc::Promise::DidNotSwap::Reason reason) {
switch (reason) {
- case cc::SwapPromise::DID_NOT_SWAP_UNKNOWN:
- case cc::SwapPromise::SWAP_FAILS:
+ case cc::Promise::DidNotSwap::UNKNOWN:
+ case cc::Promise::DidNotSwap::SWAP_FAILS:
return ui::INPUT_EVENT_LATENCY_TERMINATED_SWAP_FAILED_COMPONENT;
- case cc::SwapPromise::COMMIT_FAILS:
+ case cc::Promise::DidNotSwap::COMMIT_FAILS:
return ui::INPUT_EVENT_LATENCY_TERMINATED_COMMIT_FAILED_COMPONENT;
- case cc::SwapPromise::COMMIT_NO_UPDATE:
+ case cc::Promise::DidNotSwap::COMMIT_NO_UPDATE:
return ui::INPUT_EVENT_LATENCY_TERMINATED_COMMIT_NO_UPDATE_COMPONENT;
}
NOTREACHED() << "Unhandled DidNotSwapReason.";
@@ -37,7 +37,7 @@ void LatencyInfoSwapPromise::DidSwap(CompositorFrameMetadata* metadata) {
metadata->latency_info.push_back(latency_);
}
-void LatencyInfoSwapPromise::DidNotSwap(DidNotSwapReason reason) {
+void LatencyInfoSwapPromise::DidNotSwap(DidNotSwap::Reason reason) {
latency_.AddLatencyNumber(DidNotSwapReasonToLatencyComponentType(reason), 0,
0);
// TODO(miletus): Turn this back on once per-event LatencyInfo tracking

Powered by Google App Engine
This is Rietveld 408576698