Bug 1000287 - Remove unnecessary log lines from 'GLController'. r=kats

This commit is contained in:
Alex Deaconu 2014-04-24 17:33:14 +03:00
parent c9a341617d
commit 435b9006b6

View File

@ -126,7 +126,6 @@ public class GLController {
synchronized void serverSurfaceDestroyed() {
ThreadUtils.assertOnUiThread();
Log.w(LOGTAG, "GLController::serverSurfaceDestroyed() with mCompositorCreated=" + mCompositorCreated);
mServerSurfaceValid = false;
@ -146,12 +145,10 @@ public class GLController {
if (mCompositorCreated) {
GeckoAppShell.sendEventToGeckoSync(GeckoEvent.createCompositorPauseEvent());
}
Log.w(LOGTAG, "done GLController::serverSurfaceDestroyed()");
}
synchronized void serverSurfaceChanged(int newWidth, int newHeight) {
ThreadUtils.assertOnUiThread();
Log.w(LOGTAG, "GLController::serverSurfaceChanged(" + newWidth + ", " + newHeight + ")");
mWidth = newWidth;
mHeight = newHeight;
@ -172,14 +169,12 @@ public class GLController {
void updateCompositor() {
ThreadUtils.assertOnUiThread();
Log.w(LOGTAG, "GLController::updateCompositor with mCompositorCreated=" + mCompositorCreated);
if (mCompositorCreated) {
// If the compositor has already been created, just resume it instead. We don't need
// to block here because if the surface is destroyed before the compositor grabs it,
// we can handle that gracefully (i.e. the compositor will remain paused).
resumeCompositor(mWidth, mHeight);
Log.w(LOGTAG, "done GLController::updateCompositor with compositor resume");
return;
}
@ -194,11 +189,9 @@ public class GLController {
if (GeckoThread.checkLaunchState(GeckoThread.LaunchState.GeckoRunning)) {
GeckoAppShell.sendEventToGeckoSync(GeckoEvent.createCompositorCreateEvent(mWidth, mHeight));
}
Log.w(LOGTAG, "done GLController::updateCompositor");
}
void compositorCreated() {
Log.w(LOGTAG, "GLController::compositorCreated");
// This is invoked on the compositor thread, while the java UI thread
// is blocked on the gecko sync event in updateCompositor() above
mCompositorCreated = true;
@ -329,7 +322,6 @@ public class GLController {
}
void resumeCompositor(int width, int height) {
Log.w(LOGTAG, "GLController::resumeCompositor(" + width + ", " + height + ") and mCompositorCreated=" + mCompositorCreated);
// Asking Gecko to resume the compositor takes too long (see
// https://bugzilla.mozilla.org/show_bug.cgi?id=735230#c23), so we
// resume the compositor directly. We still need to inform Gecko about
@ -340,7 +332,6 @@ public class GLController {
GeckoAppShell.scheduleResumeComposition(width, height);
GeckoAppShell.sendEventToGecko(GeckoEvent.createCompositorResumeEvent());
}
Log.w(LOGTAG, "done GLController::resumeCompositor");
}
public static class GLControllerException extends RuntimeException {