Fix double lock of texture resource in software renderer

cc/trees/layer_tree_host_pixeltest_readback.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 "build/build_config.h"
 #include "cc/layers/solid_color_layer.h"
 #include "cc/layers/texture_layer.h"
 #include "cc/output/copy_output_request.h"
 #include "cc/output/copy_output_result.h"
 #include "cc/test/fake_picture_layer.h"
@@ skipping 1126 matching lines @@
   green->AddChild(blue);
 
   RunReadbackTestWithReadbackTarget(
       PIXEL_TEST_GL,
       READBACK_DEFAULT,
       background,
       green.get(),
       base::FilePath(FILE_PATH_LITERAL("green_with_blue_corner.png")));
 }
 
+class LayerTreeHostReadbackNonFirstNonRootRenderPassTest

[danakj, 2015/02/07 00:12:57]

Sorry, last thing. It should be ....NonRootRenderPassPixelTest. We suffix all
pixel tests with that.

+    : public LayerTreeHostReadbackPixelTest,
+      public testing::WithParamInterface<bool> {};
+
+TEST_P(LayerTreeHostReadbackNonFirstNonRootRenderPassTest,
+       ReadbackNonRootOrFirstLayer) {
+  // This test has 3 render passes with the copy request on the render pass in
+  // the middle. Doing a copy request can be destructive of state, so for render
+  // passes after the first drawn the code path is different. This verifies the
+  // non-first and non-root path. See http://crbug.com/99393 for more info.
+  scoped_refptr<SolidColorLayer> background =
+      CreateSolidColorLayer(gfx::Rect(200, 200), SK_ColorGREEN);
+
+  scoped_refptr<SolidColorLayer> blue =
+      CreateSolidColorLayer(gfx::Rect(150, 150, 50, 50), SK_ColorBLUE);
+  blue->RequestCopyOfOutput(CopyOutputRequest::CreateBitmapRequest(
+      base::Bind(&IgnoreReadbackResult)));
+  background->AddChild(blue);
+
+  RunReadbackTestWithReadbackTarget(
+      GetParam() ? PIXEL_TEST_GL : PIXEL_TEST_SOFTWARE,
+      READBACK_DEFAULT,
+      background,
+      background.get(),
+      base::FilePath(FILE_PATH_LITERAL("green_with_blue_corner.png")));
+}
+
+INSTANTIATE_TEST_CASE_P(LayerTreeHostReadbackNonFirstNonRootRenderPassTests,
+                        LayerTreeHostReadbackNonFirstNonRootRenderPassTest,
+                        testing::Bool());
+
 }  // namespace
 }  // namespace cc
 
 #endif  // OS_ANDROID