Do not send SetClientHasMemoryAllocationChanged after error.

content/common/gpu/client/command_buffer_proxy_impl.cc

Index: content/common/gpu/client/command_buffer_proxy_impl.cc
diff --git a/content/common/gpu/client/command_buffer_proxy_impl.cc b/content/common/gpu/client/command_buffer_proxy_impl.cc
index 58ca3af7e3c6a41592f1ef909410122d1d665135..c08163d59e6a222bd6ab6198b51978fcc112668e 100644
--- a/content/common/gpu/client/command_buffer_proxy_impl.cc
+++ b/content/common/gpu/client/command_buffer_proxy_impl.cc
@@ -105,8 +105,9 @@ void CommandBufferProxyImpl::SetMemoryAllocationChangedCallback(
     const base::Callback<void(const GpuMemoryAllocationForRenderer&)>&
         callback) {
   memory_allocation_changed_callback_ = callback;
-  Send(new GpuCommandBufferMsg_SetClientHasMemoryAllocationChangedCallback(
-      route_id_, !memory_allocation_changed_callback_.is_null()));
+  if (last_state_.error == gpu::error::kNoError)

[Ami GONE FROM CHROMIUM, 2012/05/06 23:03:12]

Multi-line if tests or bodies require braces around the body.  But if you
reverse the test and early-return (e.g. like
CommandBufferProxyImpl::SetSurfaceVisible and many other methods in this class)
then you don't need the braces or the extra indent.

+    Send(new GpuCommandBufferMsg_SetClientHasMemoryAllocationChangedCallback(
+        route_id_, !memory_allocation_changed_callback_.is_null()));
 }
 
 void CommandBufferProxyImpl::OnSetMemoryAllocation(