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

Side by Side Diff: chrome/browser/android/offline_pages/offline_page_utils_unittest.cc

Issue 2040573002: [Offlining] Updates the tab helper to use the async OfflinePageModel API (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address jianli's nits. Created 4 years, 6 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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/android/offline_pages/offline_page_utils.h" 5 #include "chrome/browser/android/offline_pages/offline_page_utils.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/callback.h" 10 #include "base/callback.h"
(...skipping 170 matching lines...) Expand 10 before | Expand all | Expand 10 after
181 // URL is invalid. 181 // URL is invalid.
182 EXPECT_FALSE(OfflinePageUtils::MightBeOfflineURL(GURL("/test.mhtml"))); 182 EXPECT_FALSE(OfflinePageUtils::MightBeOfflineURL(GURL("/test.mhtml")));
183 // Scheme is not file. 183 // Scheme is not file.
184 EXPECT_FALSE(OfflinePageUtils::MightBeOfflineURL(GURL("http://test.com/"))); 184 EXPECT_FALSE(OfflinePageUtils::MightBeOfflineURL(GURL("http://test.com/")));
185 // Does not end with .mhtml. 185 // Does not end with .mhtml.
186 EXPECT_FALSE(OfflinePageUtils::MightBeOfflineURL(GURL("file:///test.txt"))); 186 EXPECT_FALSE(OfflinePageUtils::MightBeOfflineURL(GURL("file:///test.txt")));
187 // Might still be an offline page. 187 // Might still be an offline page.
188 EXPECT_TRUE(OfflinePageUtils::MightBeOfflineURL(GURL("file:///test.mhtml"))); 188 EXPECT_TRUE(OfflinePageUtils::MightBeOfflineURL(GURL("file:///test.mhtml")));
189 } 189 }
190 190
191 TEST_F(OfflinePageUtilsTest, GetOfflineURLForOnlineURL) { 191 TEST_F(OfflinePageUtilsTest, MaybeGetOfflineURLForOnlineURL) {
192 EXPECT_EQ(offline_url_page_1(), OfflinePageUtils::GetOfflineURLForOnlineURL( 192 EXPECT_EQ(offline_url_page_1(),
193 profile(), kTestPage1Url)); 193 OfflinePageUtils::MaybeGetOfflineURLForOnlineURL(profile(),
194 EXPECT_EQ(offline_url_page_2(), OfflinePageUtils::GetOfflineURLForOnlineURL( 194 kTestPage1Url));
195 profile(), kTestPage2Url)); 195 EXPECT_EQ(offline_url_page_2(),
196 EXPECT_EQ(GURL(), OfflinePageUtils::GetOfflineURLForOnlineURL( 196 OfflinePageUtils::MaybeGetOfflineURLForOnlineURL(profile(),
197 kTestPage2Url));
198 EXPECT_EQ(GURL(), OfflinePageUtils::MaybeGetOfflineURLForOnlineURL(
197 profile(), GURL(kTestPage3Url))); 199 profile(), GURL(kTestPage3Url)));
198 } 200 }
199 201
200 TEST_F(OfflinePageUtilsTest, GetOnlineURLForOfflineURL) { 202 TEST_F(OfflinePageUtilsTest, MaybeGetOnlineURLForOfflineURL) {
201 EXPECT_EQ(kTestPage1Url, OfflinePageUtils::GetOnlineURLForOfflineURL( 203 EXPECT_EQ(kTestPage1Url, OfflinePageUtils::MaybeGetOnlineURLForOfflineURL(
202 profile(), offline_url_page_1())); 204 profile(), offline_url_page_1()));
203 EXPECT_EQ(kTestPage2Url, OfflinePageUtils::GetOnlineURLForOfflineURL( 205 EXPECT_EQ(kTestPage2Url, OfflinePageUtils::MaybeGetOnlineURLForOfflineURL(
204 profile(), offline_url_page_2())); 206 profile(), offline_url_page_2()));
205 EXPECT_EQ(GURL::EmptyGURL(), OfflinePageUtils::GetOnlineURLForOfflineURL( 207 EXPECT_EQ(GURL::EmptyGURL(), OfflinePageUtils::MaybeGetOnlineURLForOfflineURL(
206 profile(), offline_url_missing())); 208 profile(), offline_url_missing()));
207 } 209 }
208 210
209 TEST_F(OfflinePageUtilsTest, IsOfflinePage) { 211 TEST_F(OfflinePageUtilsTest, IsOfflinePage) {
210 EXPECT_TRUE(OfflinePageUtils::IsOfflinePage(profile(), offline_url_page_1())); 212 EXPECT_TRUE(OfflinePageUtils::IsOfflinePage(profile(), offline_url_page_1()));
211 EXPECT_TRUE(OfflinePageUtils::IsOfflinePage(profile(), offline_url_page_2())); 213 EXPECT_TRUE(OfflinePageUtils::IsOfflinePage(profile(), offline_url_page_2()));
212 EXPECT_FALSE( 214 EXPECT_FALSE(
213 OfflinePageUtils::IsOfflinePage(profile(), offline_url_missing())); 215 OfflinePageUtils::IsOfflinePage(profile(), offline_url_missing()));
214 EXPECT_FALSE(OfflinePageUtils::IsOfflinePage(profile(), kTestPage1Url)); 216 EXPECT_FALSE(OfflinePageUtils::IsOfflinePage(profile(), kTestPage1Url));
215 EXPECT_FALSE(OfflinePageUtils::IsOfflinePage(profile(), kTestPage2Url)); 217 EXPECT_FALSE(OfflinePageUtils::IsOfflinePage(profile(), kTestPage2Url));
216 } 218 }
217 219
218 TEST_F(OfflinePageUtilsTest, HasOfflinePageForOnlineURL) { 220 TEST_F(OfflinePageUtilsTest, HasOfflinePageForOnlineURL) {
219 EXPECT_TRUE( 221 EXPECT_TRUE(
220 OfflinePageUtils::HasOfflinePageForOnlineURL(profile(), kTestPage1Url)); 222 OfflinePageUtils::HasOfflinePageForOnlineURL(profile(), kTestPage1Url));
221 EXPECT_TRUE( 223 EXPECT_TRUE(
222 OfflinePageUtils::HasOfflinePageForOnlineURL(profile(), kTestPage2Url)); 224 OfflinePageUtils::HasOfflinePageForOnlineURL(profile(), kTestPage2Url));
223 EXPECT_FALSE( 225 EXPECT_FALSE(
224 OfflinePageUtils::HasOfflinePageForOnlineURL(profile(), kTestPage3Url)); 226 OfflinePageUtils::HasOfflinePageForOnlineURL(profile(), kTestPage3Url));
225 } 227 }
226 228
227 } // namespace offline_pages 229 } // namespace offline_pages
OLDNEW
« no previous file with comments | « chrome/browser/android/offline_pages/offline_page_utils.cc ('k') | chrome/browser/android/tab_android.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698