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

Unified Diff: ios/chrome/browser/ui/reading_list/offline_page_native_content_unittest.mm

Issue 2800673002: [iOS Reading List] Only allow offline URL when the offline file exists. (Closed)
Patch Set: Update comments Created 3 years, 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ios/chrome/browser/ui/reading_list/offline_page_native_content.mm ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/browser/ui/reading_list/offline_page_native_content_unittest.mm
diff --git a/ios/chrome/browser/ui/reading_list/offline_page_native_content_unittest.mm b/ios/chrome/browser/ui/reading_list/offline_page_native_content_unittest.mm
index a17566bc7327eb9c90e66c48f5e42c5f30dc41bb..abc5dd5ec2a8bde10ded78216b79dd16e36b7421 100644
--- a/ios/chrome/browser/ui/reading_list/offline_page_native_content_unittest.mm
+++ b/ios/chrome/browser/ui/reading_list/offline_page_native_content_unittest.mm
@@ -49,23 +49,6 @@ TEST_F(OfflinePageNativeContentTest, BasicOfflinePageTest) {
ASSERT_OCMOCK_VERIFY((OCMockObject*)loader);
}
-// Checks the OfflinePageNativeContent without virtual URL is correctly
-// initialized
-TEST_F(OfflinePageNativeContentTest, BasicOfflinePageTestWithoutVirtualURL) {
- GURL url = reading_list::OfflineURLForPath(
- base::FilePath("offline_id/page.html"), GURL(), GURL());
- id<UrlLoader> loader = [OCMockObject mockForProtocol:@protocol(UrlLoader)];
- base::scoped_nsobject<OfflinePageNativeContent> content(
- [[OfflinePageNativeContent alloc]
- initWithLoader:loader
- browserState:chrome_browser_state_.get()
- webState:web_state()
- URL:url]);
- ASSERT_EQ(url, [content url]);
- ASSERT_EQ(url, [content virtualURL]);
- ASSERT_OCMOCK_VERIFY((OCMockObject*)loader);
-}
-
// Checks that dismissing offline page restores EntryURL.
TEST_F(OfflinePageNativeContentTest, DismissOfflineContent) {
GURL offline_url("http://foo.bar/offline");
« no previous file with comments | « ios/chrome/browser/ui/reading_list/offline_page_native_content.mm ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698