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

Unified Diff: components/offline_pages/offline_page_model_impl_unittest.cc

Issue 2260443003: [Offline Pages] Clean up some constant value types for proposed_offline_id arg. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merge Created 4 years, 4 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 | « components/offline_pages/offline_page_model_impl.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/offline_pages/offline_page_model_impl_unittest.cc
diff --git a/components/offline_pages/offline_page_model_impl_unittest.cc b/components/offline_pages/offline_page_model_impl_unittest.cc
index 39dda8e40b89f08591f23fb9dcbdef64a16b061a..722e7743eebe05925f06052134bd888c459f560b 100644
--- a/components/offline_pages/offline_page_model_impl_unittest.cc
+++ b/components/offline_pages/offline_page_model_impl_unittest.cc
@@ -324,7 +324,7 @@ void OfflinePageModelImplTest::SavePageWithArchiverResult(
OfflinePageArchiver::ArchiverResult result) {
std::unique_ptr<OfflinePageTestArchiver> archiver(BuildArchiver(url, result));
model()->SavePage(
- url, client_id, 0ul, std::move(archiver),
+ url, client_id, 0l, std::move(archiver),
base::Bind(&OfflinePageModelImplTest::OnSavePageDone, AsWeakPtr()));
PumpLoop();
}
@@ -484,7 +484,7 @@ TEST_F(OfflinePageModelImplTest, SavePageOfflineArchiverReturnedWrongUrl) {
BuildArchiver(GURL("http://other.random.url.com"),
OfflinePageArchiver::ArchiverResult::SUCCESSFULLY_CREATED));
model()->SavePage(
- kTestUrl, kTestClientId1, 0ul, std::move(archiver),
+ kTestUrl, kTestClientId1, 0l, std::move(archiver),
base::Bind(&OfflinePageModelImplTest::OnSavePageDone, AsWeakPtr()));
PumpLoop();
EXPECT_EQ(SavePageResult::ARCHIVE_CREATION_FAILED, last_save_result());
@@ -501,7 +501,7 @@ TEST_F(OfflinePageModelImplTest, SavePageLocalFileFailed) {
// Don't create archiver since it will not be needed for pages that are not
// going to be saved.
model()->SavePage(
- kFileUrl, kTestClientId1, 0ul, std::unique_ptr<OfflinePageTestArchiver>(),
+ kFileUrl, kTestClientId1, 0l, std::unique_ptr<OfflinePageTestArchiver>(),
base::Bind(&OfflinePageModelImplTest::OnSavePageDone, AsWeakPtr()));
PumpLoop();
EXPECT_EQ(SavePageResult::SKIPPED, last_save_result());
@@ -515,7 +515,7 @@ TEST_F(OfflinePageModelImplTest, SavePageOfflineArchiverTwoPages) {
OfflinePageTestArchiver* archiver_ptr = archiver.get();
archiver_ptr->set_delayed(true);
model()->SavePage(
- kTestUrl, kTestClientId1, 0ul, std::move(archiver),
+ kTestUrl, kTestClientId1, 0l, std::move(archiver),
base::Bind(&OfflinePageModelImplTest::OnSavePageDone, AsWeakPtr()));
EXPECT_TRUE(archiver_ptr->create_archive_called());
@@ -830,7 +830,7 @@ TEST_F(OfflinePageModelImplTest, DeleteMultiplePages) {
std::unique_ptr<OfflinePageTestArchiver> archiver(BuildArchiver(
kTestUrl, OfflinePageArchiver::ArchiverResult::SUCCESSFULLY_CREATED));
model()->SavePage(
- kTestUrl, kTestClientId1, 0ul, std::move(archiver),
+ kTestUrl, kTestClientId1, 0l, std::move(archiver),
base::Bind(&OfflinePageModelImplTest::OnSavePageDone, AsWeakPtr()));
PumpLoop();
int64_t offline1 = last_save_offline_id();
@@ -838,7 +838,7 @@ TEST_F(OfflinePageModelImplTest, DeleteMultiplePages) {
std::unique_ptr<OfflinePageTestArchiver> archiver2(BuildArchiver(
kTestUrl2, OfflinePageArchiver::ArchiverResult::SUCCESSFULLY_CREATED));
model()->SavePage(
- kTestUrl2, kTestClientId2, 0ul, std::move(archiver2),
+ kTestUrl2, kTestClientId2, 0l, std::move(archiver2),
base::Bind(&OfflinePageModelImplTest::OnSavePageDone, AsWeakPtr()));
PumpLoop();
int64_t offline2 = last_save_offline_id();
@@ -846,7 +846,7 @@ TEST_F(OfflinePageModelImplTest, DeleteMultiplePages) {
std::unique_ptr<OfflinePageTestArchiver> archiver3(BuildArchiver(
kTestUrl3, OfflinePageArchiver::ArchiverResult::SUCCESSFULLY_CREATED));
model()->SavePage(
- kTestUrl3, kTestClientId3, 0ul, std::move(archiver3),
+ kTestUrl3, kTestClientId3, 0l, std::move(archiver3),
base::Bind(&OfflinePageModelImplTest::OnSavePageDone, AsWeakPtr()));
PumpLoop();
« no previous file with comments | « components/offline_pages/offline_page_model_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698