Add user data dir field to Mac platform apps

chrome/browser/web_applications/web_app_unittest.cc

 // Copyright (c) 2011 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/web_applications/web_app.h"
 
 #include "base/file_path.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/ui/tab_contents/test_tab_contents_wrapper.h"
@@ skipping 26 matching lines @@
   rvh()->TestOnMessageReceived(
       ExtensionHostMsg_DidGetApplicationInfo(0, 0, web_app_info));
   ShellIntegration::ShortcutInfo info;
   web_app::GetShortcutInfoForTab(contents_wrapper(), &info);
 
   EXPECT_EQ(title, info.title);
   EXPECT_EQ(description, info.description);
   EXPECT_EQ(url, info.url);
 }
 
-TEST_F(WebApplicationTest, GetDataDir) {
-  FilePath test_path(FILE_PATH_LITERAL("/path/to/test"));
-  FilePath result = web_app::GetDataDir(test_path);
-  test_path = test_path.AppendASCII("Web Applications");
-  EXPECT_EQ(test_path.value(), result.value());
+TEST_F(WebApplicationTest, AppDirWithID) {

[Mihai Parparita -not on Chrome, 2012/02/22 23:24:15]

Nit: This should be called AppDirWithId.

[sail, 2012/02/22 23:38:48]

Done.

+  FilePath profile_path(FILE_PATH_LITERAL("profile"));
+  FilePath result(web_app::GetWebAppDataDirectory(profile_path, "123", GURL()));
+  FilePath expected = profile_path.AppendASCII("Web Applications")
+                                  .AppendASCII("_crx_123");
+  EXPECT_EQ(expected, result);
 }
+
+TEST_F(WebApplicationTest, AppDirWithURL) {

[Mihai Parparita -not on Chrome, 2012/02/22 23:24:15]

Ditto about AppDirWithUrl.

[sail, 2012/02/22 23:38:48]

Done.

+  FilePath profile_path(FILE_PATH_LITERAL("profile"));
+  FilePath result(web_app::GetWebAppDataDirectory(
+      profile_path, "", GURL("http://example.com")));
+  FilePath expected = profile_path.AppendASCII("Web Applications")
+                                  .AppendASCII("example.com")
+                                  .AppendASCII("http_80");
+  EXPECT_EQ(expected, result);
+}