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

Side by Side Diff: components/offline_pages/offline_page_model_impl.cc

Issue 2083653008: Temporarily remove "last_n" from availability searches. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: gNit. 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "components/offline_pages/offline_page_model_impl.h" 5 #include "components/offline_pages/offline_page_model_impl.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <limits> 8 #include <limits>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
11 #include "base/files/file_util.h" 11 #include "base/files/file_util.h"
12 #include "base/location.h" 12 #include "base/location.h"
13 #include "base/logging.h" 13 #include "base/logging.h"
14 #include "base/metrics/histogram_macros.h" 14 #include "base/metrics/histogram_macros.h"
15 #include "base/rand_util.h" 15 #include "base/rand_util.h"
16 #include "base/sequenced_task_runner.h" 16 #include "base/sequenced_task_runner.h"
17 #include "base/strings/string_number_conversions.h" 17 #include "base/strings/string_number_conversions.h"
18 #include "base/threading/thread_task_runner_handle.h" 18 #include "base/threading/thread_task_runner_handle.h"
19 #include "base/time/time.h" 19 #include "base/time/time.h"
20 #include "components/offline_pages/archive_manager.h" 20 #include "components/offline_pages/archive_manager.h"
21 #include "components/offline_pages/client_namespace_constants.h"
21 #include "components/offline_pages/client_policy_controller.h" 22 #include "components/offline_pages/client_policy_controller.h"
22 #include "components/offline_pages/offline_page_item.h" 23 #include "components/offline_pages/offline_page_item.h"
23 #include "components/offline_pages/offline_page_storage_manager.h" 24 #include "components/offline_pages/offline_page_storage_manager.h"
24 #include "url/gurl.h" 25 #include "url/gurl.h"
25 26
26 using ArchiverResult = offline_pages::OfflinePageArchiver::ArchiverResult; 27 using ArchiverResult = offline_pages::OfflinePageArchiver::ArchiverResult;
27 using ClearStorageCallback = 28 using ClearStorageCallback =
28 offline_pages::OfflinePageStorageManager::ClearStorageCallback; 29 offline_pages::OfflinePageStorageManager::ClearStorageCallback;
29 using ClearStorageResult = 30 using ClearStorageResult =
30 offline_pages::OfflinePageStorageManager::ClearStorageResult; 31 offline_pages::OfflinePageStorageManager::ClearStorageResult;
(...skipping 399 matching lines...) Expand 10 before | Expand all | Expand 10 after
430 base::Bind(&OfflinePageModelImpl::CheckPagesExistOfflineAfterLoadDone, 431 base::Bind(&OfflinePageModelImpl::CheckPagesExistOfflineAfterLoadDone,
431 weak_ptr_factory_.GetWeakPtr(), urls, callback)); 432 weak_ptr_factory_.GetWeakPtr(), urls, callback));
432 } 433 }
433 434
434 void OfflinePageModelImpl::CheckPagesExistOfflineAfterLoadDone( 435 void OfflinePageModelImpl::CheckPagesExistOfflineAfterLoadDone(
435 const std::set<GURL>& urls, 436 const std::set<GURL>& urls,
436 const CheckPagesExistOfflineCallback& callback) { 437 const CheckPagesExistOfflineCallback& callback) {
437 DCHECK(is_loaded_); 438 DCHECK(is_loaded_);
438 CheckPagesExistOfflineResult result; 439 CheckPagesExistOfflineResult result;
439 for (const auto& id_page_pair : offline_pages_) { 440 for (const auto& id_page_pair : offline_pages_) {
440 if (id_page_pair.second.IsExpired()) 441 // TODO(dewittj): Remove the "Last N" restriction in favor of a better query
442 // interface. See https://crbug.com/622763 for information.
443 if (id_page_pair.second.IsExpired() ||
444 id_page_pair.second.client_id.name_space == kLastNNamespace)
441 continue; 445 continue;
442 auto iter = urls.find(id_page_pair.second.url); 446 auto iter = urls.find(id_page_pair.second.url);
443 if (iter != urls.end()) 447 if (iter != urls.end())
444 result.insert(*iter); 448 result.insert(*iter);
445 } 449 }
446 callback.Run(result); 450 callback.Run(result);
447 } 451 }
448 452
449 void OfflinePageModelImpl::GetAllPages( 453 void OfflinePageModelImpl::GetAllPages(
450 const MultipleOfflinePageItemCallback& callback) { 454 const MultipleOfflinePageItemCallback& callback) {
(...skipping 532 matching lines...) Expand 10 before | Expand all | Expand 10 after
983 void OfflinePageModelImpl::RunWhenLoaded(const base::Closure& task) { 987 void OfflinePageModelImpl::RunWhenLoaded(const base::Closure& task) {
984 if (!is_loaded_) { 988 if (!is_loaded_) {
985 delayed_tasks_.push_back(task); 989 delayed_tasks_.push_back(task);
986 return; 990 return;
987 } 991 }
988 992
989 base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE, task); 993 base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE, task);
990 } 994 }
991 995
992 } // namespace offline_pages 996 } // namespace offline_pages
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698