[Extensions] Fix lifetime bug in ExtensionAction/IconImage

chrome/browser/ui/toolbar/browser_actions_bar_browsertest.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/toolbar/browser_actions_bar_browsertest.h"
 
 #include <stddef.h>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/macros.h"
 #include "base/run_loop.h"
+#include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/extensions/api/extension_action/extension_action_api.h"
 #include "chrome/browser/extensions/browser_action_test_util.h"
 #include "chrome/browser/extensions/extension_action.h"
 #include "chrome/browser/extensions/extension_action_manager.h"
 #include "chrome/browser/extensions/extension_action_test_util.h"
 #include "chrome/browser/extensions/extension_browsertest.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/sessions/session_tab_helper.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/extensions/extension_action_view_controller.h"
@@ skipping 606 matching lines @@
   EXPECT_EQ(2, browser_actions_bar()->VisibleBrowserActions());
   ASSERT_TRUE(toolbar_actions_bar->popped_out_action());
   EXPECT_EQ(extension3->id(),
             toolbar_actions_bar->popped_out_action()->GetId());
   extension_service()->UnloadExtension(
       extension3->id(), extensions::UnloadedExtensionInfo::REASON_DISABLE);
   EXPECT_EQ(1, browser_actions_bar()->VisibleBrowserActions());
   EXPECT_EQ(1, browser_actions_bar()->NumberOfBrowserActions());
   EXPECT_FALSE(toolbar_actions_bar->popped_out_action());
 }
+
+// A test that runs in incognito mode.
+class BrowserActionsBarIncognitoTest : public BrowserActionsBarBrowserTest {
+ public:
+  void SetUpCommandLine(base::CommandLine* command_line) override {
+    BrowserActionsBarBrowserTest::SetUpCommandLine(command_line);
+    command_line->AppendSwitch("incognito");
+  }
+};
+
+// Tests that first loading an extension action in an incognito profile, then
+// removing the incognito profile and using the extension action in a normal
+// profile doesn't crash.
+// Regression test for crbug.com/663726.
+IN_PROC_BROWSER_TEST_F(BrowserActionsBarIncognitoTest, IncognitoMode) {
+  EXPECT_TRUE(browser()->profile()->IsOffTheRecord());
+  const extensions::Extension* extension = LoadExtensionIncognito(
+      test_data_dir_.AppendASCII("api_test/browser_action_with_icon"));
+  ASSERT_TRUE(extension);
+  Browser* second_browser =
+      new Browser(Browser::CreateParams(profile()->GetOriginalProfile()));
+  base::RunLoop().RunUntilIdle();
+  EXPECT_FALSE(second_browser->profile()->IsOffTheRecord());
+
+  content::WindowedNotificationObserver window_close_observer(
+      chrome::NOTIFICATION_BROWSER_CLOSED, content::Source<Browser>(browser()));
+  browser()->window()->Close();
+  window_close_observer.Wait();
+
+  std::vector<ToolbarActionViewController*> actions =
+      second_browser->window()->GetToolbarActionsBar()->GetActions();
+  ASSERT_EQ(1u, actions.size());
+  gfx::Image icon = actions[0]->GetIcon(
+      second_browser->tab_strip_model()->GetActiveWebContents(),
+      gfx::Size(ToolbarActionsBar::IconWidth(false),
+                ToolbarActionsBar::IconHeight()));
+  const gfx::ImageSkia* skia = icon.ToImageSkia();
+  ASSERT_TRUE(skia);
+  // Force the image to try and load a representation.
+  skia->GetRepresentation(2.0);
+}