Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(114)

Side by Side Diff: chrome/browser/web_applications/web_app_unittest.cc

Issue 568823003: Merge Android RetrieveWebappInformation and Extensions GetApplicationInfo. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@manifest_manager_content
Patch Set: Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « chrome/browser/extensions/tab_helper.cc ('k') | chrome/chrome_renderer.gypi » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/web_applications/web_app.h" 5 #include "chrome/browser/web_applications/web_app.h"
6 6
7 #include "base/files/file_path.h" 7 #include "base/files/file_path.h"
8 #include "base/strings/string_util.h" 8 #include "base/strings/string_util.h"
9 #include "base/strings/utf_string_conversions.h" 9 #include "base/strings/utf_string_conversions.h"
10 #include "chrome/browser/web_applications/web_app.h" 10 #include "chrome/browser/web_applications/web_app.h"
11 #include "chrome/common/extensions/chrome_extension_messages.h" 11 #include "chrome/common/render_messages.h"
12 #include "chrome/test/base/chrome_render_view_host_test_harness.h" 12 #include "chrome/test/base/chrome_render_view_host_test_harness.h"
13 #include "content/public/test/test_renderer_host.h" 13 #include "content/public/test/test_renderer_host.h"
14 #include "testing/gtest/include/gtest/gtest.h" 14 #include "testing/gtest/include/gtest/gtest.h"
15 15
16 #if defined(TOOLKIT_VIEWS) 16 #if defined(TOOLKIT_VIEWS)
17 #include "chrome/browser/extensions/tab_helper.h" 17 #include "chrome/browser/extensions/tab_helper.h"
18 #include "chrome/browser/favicon/favicon_tab_helper.h" 18 #include "chrome/browser/favicon/favicon_tab_helper.h"
19 #endif 19 #endif
20 20
21 using content::RenderViewHostTester; 21 using content::RenderViewHostTester;
(...skipping 13 matching lines...) Expand all
35 TEST_F(WebApplicationTest, GetShortcutInfoForTab) { 35 TEST_F(WebApplicationTest, GetShortcutInfoForTab) {
36 const base::string16 title = base::ASCIIToUTF16("TEST_TITLE"); 36 const base::string16 title = base::ASCIIToUTF16("TEST_TITLE");
37 const base::string16 description = base::ASCIIToUTF16("TEST_DESCRIPTION"); 37 const base::string16 description = base::ASCIIToUTF16("TEST_DESCRIPTION");
38 const GURL url("http://www.foo.com/bar"); 38 const GURL url("http://www.foo.com/bar");
39 WebApplicationInfo web_app_info; 39 WebApplicationInfo web_app_info;
40 web_app_info.title = title; 40 web_app_info.title = title;
41 web_app_info.description = description; 41 web_app_info.description = description;
42 web_app_info.app_url = url; 42 web_app_info.app_url = url;
43 43
44 RenderViewHostTester::TestOnMessageReceived( 44 RenderViewHostTester::TestOnMessageReceived(
45 rvh(), ChromeExtensionHostMsg_DidGetApplicationInfo(0, web_app_info)); 45 rvh(), ChromeViewHostMsg_DidGetWebApplicationInfo(0, web_app_info));
46 web_app::ShortcutInfo info; 46 web_app::ShortcutInfo info;
47 web_app::GetShortcutInfoForTab(web_contents(), &info); 47 web_app::GetShortcutInfoForTab(web_contents(), &info);
48 48
49 EXPECT_EQ(title, info.title); 49 EXPECT_EQ(title, info.title);
50 EXPECT_EQ(description, info.description); 50 EXPECT_EQ(description, info.description);
51 EXPECT_EQ(url, info.url); 51 EXPECT_EQ(url, info.url);
52 } 52 }
53 #endif 53 #endif
54 54
55 #if defined(ENABLE_EXTENSIONS)
55 TEST_F(WebApplicationTest, AppDirWithId) { 56 TEST_F(WebApplicationTest, AppDirWithId) {
56 base::FilePath profile_path(FILE_PATH_LITERAL("profile")); 57 base::FilePath profile_path(FILE_PATH_LITERAL("profile"));
57 base::FilePath result( 58 base::FilePath result(
58 web_app::GetWebAppDataDirectory(profile_path, "123", GURL())); 59 web_app::GetWebAppDataDirectory(profile_path, "123", GURL()));
59 base::FilePath expected = profile_path.AppendASCII("Web Applications") 60 base::FilePath expected = profile_path.AppendASCII("Web Applications")
60 .AppendASCII("_crx_123"); 61 .AppendASCII("_crx_123");
61 EXPECT_EQ(expected, result); 62 EXPECT_EQ(expected, result);
62 } 63 }
63 64
64 TEST_F(WebApplicationTest, AppDirWithUrl) { 65 TEST_F(WebApplicationTest, AppDirWithUrl) {
65 base::FilePath profile_path(FILE_PATH_LITERAL("profile")); 66 base::FilePath profile_path(FILE_PATH_LITERAL("profile"));
66 base::FilePath result(web_app::GetWebAppDataDirectory( 67 base::FilePath result(web_app::GetWebAppDataDirectory(
67 profile_path, std::string(), GURL("http://example.com"))); 68 profile_path, std::string(), GURL("http://example.com")));
68 base::FilePath expected = profile_path.AppendASCII("Web Applications") 69 base::FilePath expected = profile_path.AppendASCII("Web Applications")
69 .AppendASCII("example.com").AppendASCII("http_80"); 70 .AppendASCII("example.com").AppendASCII("http_80");
70 EXPECT_EQ(expected, result); 71 EXPECT_EQ(expected, result);
71 } 72 }
73 #endif // ENABLE_EXTENSIONS
OLDNEW
« no previous file with comments | « chrome/browser/extensions/tab_helper.cc ('k') | chrome/chrome_renderer.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698