Simplify FrameConsumer interface. Remove FrameProducer interface.

remoting/client/plugin/chromoting_instance.cc

 // Copyright (c) 2012 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 "remoting/client/plugin/chromoting_instance.h"
 
 #include <string>
 #include <vector>
 
 #include <nacl_io/nacl_io.h>
@@ skipping 540 matching lines @@
   } else if (key_filter == "cros") {
     normalizing_input_filter_.reset(
         new NormalizingInputFilterCros(&key_mapper_));
   } else {
     DCHECK(key_filter.empty());
     normalizing_input_filter_.reset(new protocol::InputFilter(&key_mapper_));
   }
   input_handler_.set_input_stub(normalizing_input_filter_.get());
 
   // Try initializing 3D video renderer.
-  video_renderer_.reset(new PepperVideoRenderer3D());
-  if (!video_renderer_->Initialize(this, context_, this))
-    video_renderer_.reset();
+  // video_renderer_.reset(new PepperVideoRenderer3D());
+  // if (!video_renderer_->Initialize(this, context_, this))
+  //   video_renderer_.reset();

[Jamie, 2015/08/19 20:52:10]

Did you leave this commented out intentionally? If so, please also add a TODO to
either reinstate or remove it.

[Sergey Ulanov, 2015/08/19 23:37:13]

No. Commented it just to test the other changes in this CL and forget to revert.

 
   // If we didn't initialize 3D renderer then use the 2D renderer.
   if (!video_renderer_) {
     LOG(WARNING)
         << "Failed to initialize 3D renderer. Using 2D renderer instead.";
     video_renderer_.reset(new PepperVideoRenderer2D());
     if (!video_renderer_->Initialize(this, context_, this))
       video_renderer_.reset();
   }
 
@@ skipping 500 matching lines @@
 
   if (is_custom_counts_histogram)
     uma.HistogramCustomCounts(histogram_name, value, histogram_min,
                               histogram_max, histogram_buckets);
   else
     uma.HistogramCustomTimes(histogram_name, value, histogram_min,
                              histogram_max, histogram_buckets);
 }
 
 }  // namespace remoting