Fix PDFBrowserTest.Basic crash.

chrome/browser/ui/pdf/pdf_browsertest_base.cc

 // Copyright 2014 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 "chrome/browser/ui/pdf/pdf_browsertest_base.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/command_line.h"
@@ skipping 115 matching lines @@
   scoped_ptr<char[]> data(new char[size]);
   ASSERT_EQ(size, base::ReadFile(reference, data.get(), size));
 
   int w, h;
   std::vector<unsigned char> decoded;
   ASSERT_TRUE(gfx::PNGCodec::Decode(
       reinterpret_cast<unsigned char*>(data.get()), size,
       gfx::PNGCodec::FORMAT_BGRA, &decoded, &w, &h));
   int32* ref_pixels = reinterpret_cast<int32*>(&decoded[0]);
 
+  SkAutoLockPixels lock_image(bitmap);
   int32* pixels = static_cast<int32*>(bitmap.getPixels());
 
   // Get the background color, and use it to figure out the x-offsets in
   // each image.  The reason is that depending on the theme in the OS, the
   // same browser width can lead to slightly different plugin sizes, so the
   // pdf content will start at different x offsets.
   // Also note that the images we saved are cut off before the scrollbar, as
   // that'll change depending on the theme, and also cut off vertically so
   // that the ui controls don't show up, as those fade-in and so the timing
   // will affect their transparency.
@@ skipping 48 matching lines @@
   // Calling RenderWidgetHost::CopyFromBackingStore() with the GPU enabled
   // fails on Linux.
   command_line->AppendSwitch(switches::kDisableGpu);
 #endif
 
 #if defined(OS_CHROMEOS)
   // Also need on CrOS in addition to disabling the GPU above.
   command_line->AppendSwitch(switches::kUIDisableThreadedCompositing);
 #endif
 }