toggle FPS counter in compositor

cc/single_thread_proxy.cc

 // Copyright 2011 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.
 
 #include "config.h"
 
 #include "CCSingleThreadProxy.h"
 
 #include "CCDrawQuad.h"
 #include "CCGraphicsContext.h"
@@ skipping 287 matching lines @@
 
 void CCSingleThreadProxy::forceSerializeOnSwapBuffers()
 {
     {
         DebugScopedSetImplThread impl;
         if (m_rendererInitialized)
             m_layerTreeHostImpl->renderer()->doNoOp();
     }
 }
 
+void CCSingleThreadProxy::setShowFPSCounter(bool show)
+{
+    ASSERT(CCProxy::isMainThread());
+    {

[brianderson, 2012/10/18 00:13:21]

I don't think this anonymous scope is necessary.

[egraether, 2012/10/18 15:56:30]

I saw that other similar code parts also don't use it. I will remove it.

On 2012/10/18 00:13:21, Brian Anderson wrote:

+      DebugScopedSetImplThread impl;
+      ASSERT(CCProxy::isImplThread());

[brianderson, 2012/10/18 00:13:21]

You should move this ASSERT into CCLayerTreeHostImpl::setShowFPSCounter.

[egraether, 2012/10/18 15:56:30]

You are right.

On 2012/10/18 00:13:21, Brian Anderson wrote:

+      if (m_rendererInitialized)
+        m_layerTreeHostImpl->setShowFPSCounter(show);
+    }
+}
+
 void CCSingleThreadProxy::onSwapBuffersCompleteOnImplThread()
 {
     ASSERT_NOT_REACHED();
 }
 
 bool CCSingleThreadProxy::commitAndComposite()
 {
     ASSERT(CCProxy::isMainThread());
 
     if (!m_layerTreeHost->initializeRendererIfNeeded())
@@ skipping 56 matching lines @@
 
 void CCSingleThreadProxy::didSwapFrame()
 {
     if (m_nextFrameIsNewlyCommittedFrame) {
         m_nextFrameIsNewlyCommittedFrame = false;
         m_layerTreeHost->didCommitAndDrawFrame();
     }
 }
 
 }