Don't report OnFirstSurfaceActivation to CompositorImpl

Bug: 893850
Change-Id: Icb5d59ac7dd6a6cf1dfd6e17f913966d6d546c29
Reviewed-on: https://chromium-review.googlesource.com/c/1294012Reviewed-by: Fady Samuel <fsamuel@chromium.org>
Commit-Queue: Saman Sami <samans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#601616}

content/browser/renderer_host/compositor_impl_android.cc

@@ -991,7 +991,7 @@ void CompositorImpl::TearDownDisplayAndUnregisterRootFrameSink() {
 
 void CompositorImpl::RegisterRootFrameSink() {
   GetHostFrameSinkManager()->RegisterFrameSinkId(
-      frame_sink_id_, this, viz::ReportFirstSurfaceActivation::kYes);
+      frame_sink_id_, this, viz::ReportFirstSurfaceActivation::kNo);
   GetHostFrameSinkManager()->SetFrameSinkDebugLabel(frame_sink_id_,
                                                     "CompositorImpl");
 }
@@ -1441,4 +1441,8 @@ void CompositorImpl::OnFatalOrSurfaceContextCreationFailure(
   }
 }
 
+void CompositorImpl::OnFirstSurfaceActivation(const viz::SurfaceInfo& info) {
+  NOTREACHED();
+}
+
 }  // namespace content

content/browser/renderer_host/compositor_impl_android.h

@@ -148,8 +148,7 @@ class CONTENT_EXPORT CompositorImpl
   void SetVSyncPaused(bool paused) override;
 
   // viz::HostFrameSinkClient implementation.
-  void OnFirstSurfaceActivation(const viz::SurfaceInfo& surface_info) override {
-  }
+  void OnFirstSurfaceActivation(const viz::SurfaceInfo& surface_info) override;
   void OnFrameTokenChanged(uint32_t frame_token) override {}
 
   // display::DisplayObserver implementation.