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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/compositor/CompositorViewHolder.java

Issue 2718903002: android: Rename SwapBuffers to SwapFrames (Closed)
Patch Set: Created 3 years, 10 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: chrome/android/java/src/org/chromium/chrome/browser/compositor/CompositorViewHolder.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/compositor/CompositorViewHolder.java b/chrome/android/java/src/org/chromium/chrome/browser/compositor/CompositorViewHolder.java
index e4c3de2f19ed3e6e771db2b15dafb49048585158..636e75bd94ba3827a7b3845647c69160660455ea 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/compositor/CompositorViewHolder.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/compositor/CompositorViewHolder.java
@@ -85,7 +85,7 @@ public class CompositorViewHolder extends FrameLayout
private boolean mContentOverlayVisiblity = true;
- private int mPendingSwapBuffersCount;
+ private int mPendingFrameCount;
private final ArrayList<Invalidator.Client> mPendingInvalidations =
new ArrayList<>();
@@ -583,13 +583,13 @@ public class CompositorViewHolder extends FrameLayout
@Override
public void onSurfaceCreated() {
- mPendingSwapBuffersCount = 0;
+ mPendingFrameCount = 0;
flushInvalidation();
}
@Override
- public void onSwapBuffersCompleted(int pendingSwapBuffersCount) {
- TraceEvent.instant("onSwapBuffersCompleted");
+ public void didSwapFrame(int pendingFrameCount) {
+ TraceEvent.instant("didSwapFrame");
// Wait until the second frame to turn off the placeholder background on
// tablets so the tab strip has time to start drawing.
@@ -605,9 +605,9 @@ public class CompositorViewHolder extends FrameLayout
mHasDrawnOnce = true;
- mPendingSwapBuffersCount = pendingSwapBuffersCount;
+ mPendingFrameCount = pendingFrameCount;
- if (!mSkipInvalidation || pendingSwapBuffersCount == 0) flushInvalidation();
+ if (!mSkipInvalidation || pendingFrameCount == 0) flushInvalidation();
mSkipInvalidation = !mSkipInvalidation;
}
@@ -953,7 +953,7 @@ public class CompositorViewHolder extends FrameLayout
@Override
public void deferInvalidate(Client client) {
- if (mPendingSwapBuffersCount <= 0) {
+ if (mPendingFrameCount <= 0) {
client.doInvalidate();
} else if (!mPendingInvalidations.contains(client)) {
mPendingInvalidations.add(client);

Powered by Google App Engine
This is Rietveld 408576698