Crash when taking screenshot onto Chrome OS Google Drive

chrome/browser/ui/ash/screenshot_taker_unittest.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 "chrome/browser/ui/ash/screenshot_taker.h"
 
 #include "ash/shell.h"
 #include "ash/test/ash_test_base.h"
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 45 matching lines @@
     screenshot_result_ = screenshot_result;
     screenshot_path_ = screenshot_path;
     if (!running_)
       return;
     message_loop_runner_->Quit();
     running_ = false;
   }
 
  protected:
   // ScreenshotTakerTest is a friend of ScreenshotTaker and therefore
-  // allowed to set the directory and basename.
+  // allowed to set the directory, basename and profile.
   void SetScreenshotDirectoryForTest(
       ScreenshotTaker* screenshot_taker,
       const base::FilePath& screenshot_directory) {
     screenshot_taker->SetScreenshotDirectoryForTest(screenshot_directory);
   }
   void SetScreenshotBasenameForTest(
       ScreenshotTaker* screenshot_taker,
       const std::string& screenshot_basename) {
     screenshot_taker->SetScreenshotBasenameForTest(screenshot_basename);
   }
+  void SetScreenshotProfileForTest(
+      ScreenshotTaker* screenshot_taker,
+      Profile* profile) {
+    screenshot_taker->SetScreenshotProfileForTest(profile);
+  }
 
   void Wait() {
     if (screenshot_complete_)
       return;
     running_ = true;
     message_loop_runner_ = new content::MessageLoopRunner;
     message_loop_runner_->Run();
     EXPECT_TRUE(screenshot_complete_);
   }
 
   scoped_ptr<ScopedTestingLocalState> local_state_;
   content::TestBrowserThread ui_thread_;
   bool running_;
   bool screenshot_complete_;
   ScreenshotTakerObserver::Result screenshot_result_;
   base::FilePath screenshot_path_;
   scoped_refptr<content::MessageLoopRunner> message_loop_runner_;
 
   DISALLOW_COPY_AND_ASSIGN(ScreenshotTakerTest);
 };
 
 TEST_F(ScreenshotTakerTest, TakeScreenshot) {
   TestingProfile profile;
-  ScreenshotTaker screenshot_taker(&profile);
+  ScreenshotTaker screenshot_taker;
   screenshot_taker.AddObserver(this);
   base::ScopedTempDir directory;
   ASSERT_TRUE(directory.CreateUniqueTempDir());
   SetScreenshotDirectoryForTest(&screenshot_taker, directory.path());
   SetScreenshotBasenameForTest(&screenshot_taker, "Screenshot");
+  SetScreenshotProfileForTest(&screenshot_taker, &profile);
 
   EXPECT_TRUE(screenshot_taker.CanTakeScreenshot());
 
   screenshot_taker.HandleTakePartialScreenshot(
       Shell::GetPrimaryRootWindow(), gfx::Rect(0, 0, 100, 100));
 
   EXPECT_FALSE(screenshot_taker.CanTakeScreenshot());
 
   Wait();
 
 #if defined(OS_CHROMEOS)
   // Screenshot notifications on Windows not yet turned on.
   EXPECT_TRUE(g_browser_process->notification_ui_manager()->DoesIdExist(
-                  std::string("screenshot_001")));
+                  std::string("screenshot")));
   g_browser_process->notification_ui_manager()->CancelAll();
 #endif
 
   EXPECT_EQ(ScreenshotTakerObserver::SCREENSHOT_SUCCESS, screenshot_result_);
 
   if (ScreenshotTakerObserver::SCREENSHOT_SUCCESS == screenshot_result_)
     EXPECT_TRUE(file_util::PathExists(screenshot_path_));
 }
 
 }  // namespace test
 }  // namespace ash