Fix crash in sync compositor registration

content/browser/android/in_process/synchronous_compositor_impl.cc

 // Copyright 2013 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 "content/browser/android/in_process/synchronous_compositor_impl.h"
 
 #include "base/auto_reset.h"
 #include "base/bind.h"
 #include "base/lazy_instance.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 256 matching lines @@
   UpdateNeedsBeginFrames();
 }
 
 void SynchronousCompositorImpl::OnNeedsBeginFramesChange(
     bool needs_begin_frames) {
   renderer_needs_begin_frames_ = needs_begin_frames;
   UpdateNeedsBeginFrames();
 }
 
 void SynchronousCompositorImpl::BeginFrame(const cc::BeginFrameArgs& args) {
-  if (!registered_with_client_) {
+  if (!registered_with_client_ && is_active_ && renderer_needs_begin_frames_) {
+    // Make sure this is a BeginFrame that renderer side explicitly requested.
+    // Otherwise it is possible renderer objects not initialized.
     RegisterWithClient();
     DCHECK(registered_with_client_);
   }
   if (begin_frame_source_)
     begin_frame_source_->BeginFrame(args);
 }
 
 void SynchronousCompositorImpl::UpdateNeedsBeginFrames() {
   RenderWidgetHostViewAndroid* rwhv = static_cast<RenderWidgetHostViewAndroid*>(
       contents_->GetRenderWidgetHostView());
@@ skipping 100 matching lines @@
     g_factory.Get();  // Ensure it's initialized.
     SynchronousCompositorImpl::CreateForWebContents(contents);
   }
   SynchronousCompositorImpl* instance =
       SynchronousCompositorImpl::FromWebContents(contents);
   DCHECK(instance);
   instance->SetClient(client);
 }
 
 }  // namespace content