cc: Fix the format of GpuMemoryBuffer for SurfaceTexture

content/renderer/render_thread_impl.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 "content/renderer/render_thread_impl.h"
 
 #include <algorithm>
 #include <limits>
 #include <map>
 #include <vector>
@@ skipping 399 matching lines @@
 
  private:
   mojo::StrongBinding<EmbeddedWorkerSetup> binding_;
 };
 
 void CreateEmbeddedWorkerSetup(
     mojo::InterfaceRequest<EmbeddedWorkerSetup> request) {
   new EmbeddedWorkerSetupImpl(request.Pass());
 }
 
+void StringToUintVector(const std::string& str, std::vector<uint>* vector) {
+  DCHECK(vector);

[reveman, 2015/07/27 17:07:15]

This DCHECK seems overkill. Can you instead DCHECK that the vector is empty as
we assume that when using push_back below?

+  std::vector<std::string> pieces;
+  base::SplitString(str, ',', &pieces);
+  DCHECK(pieces.size() == gfx::GpuMemoryBuffer::FORMAT_LAST + 1);

[reveman, 2015/07/27 17:07:15]

nit: use DCHECK_EQ

+  for (size_t i = 0; i < pieces.size(); ++i) {
+    uint number = 0;
+    bool succeed = base::StringToUint(pieces[i], &number);
+    DCHECK(succeed);
+    vector->push_back(number);
+  }
+}
+
 }  // namespace
 
 class RasterWorkerPool : public base::SequencedTaskRunner,
                          public base::DelegateSimpleThread::Delegate {
  public:
   RasterWorkerPool()
       : namespace_token_(task_graph_runner_.GetNamespaceToken()) {}
 
   void Start(int num_threads,
              const base::SimpleThread::Options& thread_options) {
@@ skipping 328 matching lines @@
         key, kCFPreferencesCurrentApplication, &key_exists);
     if (key_exists && !value)
       is_elastic_overscroll_enabled_ = false;
   }
 #else
   is_elastic_overscroll_enabled_ = false;
 #endif
 
   std::string image_texture_target_string =
       command_line.GetSwitchValueASCII(switches::kContentImageTextureTarget);
-  bool parsed_image_texture_target = base::StringToUint(
-      image_texture_target_string, &use_image_texture_target_);
-  DCHECK(parsed_image_texture_target);
+  StringToUintVector(image_texture_target_string, &use_image_texture_targets_);
 
   if (command_line.HasSwitch(switches::kDisableLCDText)) {
     is_lcd_text_enabled_ = false;
   } else if (command_line.HasSwitch(switches::kEnableLCDText)) {
     is_lcd_text_enabled_ = true;
   } else {
 #if defined(OS_ANDROID)
     is_lcd_text_enabled_ = false;
 #else
     is_lcd_text_enabled_ = true;
@@ skipping 784 matching lines @@
 }
 
 bool RenderThreadImpl::IsOneCopyEnabled() {
   return is_one_copy_enabled_;
 }
 
 bool RenderThreadImpl::IsElasticOverscrollEnabled() {
   return is_elastic_overscroll_enabled_;
 }
 
-uint32 RenderThreadImpl::GetImageTextureTarget() {
-  return use_image_texture_target_;
+std::vector<uint> RenderThreadImpl::GetImageTextureTargets() {
+  return use_image_texture_targets_;
 }
 
 scoped_refptr<base::SingleThreadTaskRunner>
 RenderThreadImpl::GetCompositorMainThreadTaskRunner() {
   return main_thread_compositor_task_runner_;
 }
 
 scoped_refptr<base::SingleThreadTaskRunner>
 RenderThreadImpl::GetCompositorImplThreadTaskRunner() {
   return compositor_task_runner_;
@@ skipping 450 matching lines @@
 }
 
 void RenderThreadImpl::PendingRenderFrameConnect::OnConnectionError() {
   size_t erased =
       RenderThreadImpl::current()->pending_render_frame_connects_.erase(
           routing_id_);
   DCHECK_EQ(1u, erased);
 }
 
 }  // namespace content