Get ride of chrome.storage.* data when removing an application.

chrome/browser/apps/app_browsertest.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 "apps/launcher.h"
 #include "apps/shell_window.h"
 #include "apps/shell_window_registry.h"
 #include "apps/ui/native_app_window.h"
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 1274 matching lines @@
   launched_listener.Reply("restart");
   ExtensionTestMessageListener restart_requested_listener("restartRequested",
                                                           false);
   ASSERT_TRUE(restart_requested_listener.WaitUntilSatisfied());
 
   EXPECT_EQ(1, request_restart_call_count());
 }
 
 #endif  // defined(OS_CHROMEOS)
 
+// Test that when an application is uninstalled and re-install it does not have
+// access to the previously set data.
+IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, ReinstallDataCleanup) {
+  // The application is installed and launched. After the 'Launched' message is
+  // acknowledged by the browser process, the application will test that some
+  // data are not installed and then install them. The application will then be
+  // uninstalled and the same process will be repeated.
+  ExtensionTestMessageListener launched_listener("Launched", false);
+  const Extension* extension =
+      LoadAndLaunchPlatformApp("reinstall_data_cleanup");
+  ASSERT_TRUE(extension);
+
+  ExtensionApiTest::ResultCatcher result_catcher;
+  ASSERT_TRUE(launched_listener.WaitUntilSatisfied());
+
+  EXPECT_TRUE(result_catcher.GetNextResult());
+
+  UninstallExtension(extension->id());
+  content::RunAllPendingInMessageLoop();
+
+  {
+    ExtensionTestMessageListener launched_listener("Launched", false);

[not at google - send to devlin, 2014/01/08 02:49:17]

perhaps give the first half of the test the {} scoping?

[mlamouri (slow - plz ping), 2014/01/08 17:42:31]

It does not provide much advantage but why not.

+    LoadAndLaunchPlatformApp("reinstall_data_cleanup");

[not at google - send to devlin, 2014/01/08 02:49:17]

assert that this gets the same ID as before

[mlamouri (slow - plz ping), 2014/01/08 17:42:31]

OK.

+
+    ExtensionApiTest::ResultCatcher result_catcher;
+
+    ASSERT_TRUE(launched_listener.WaitUntilSatisfied());
+
+    EXPECT_TRUE(result_catcher.GetNextResult());
+  }
+}
 
 }  // namespace extensions