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

Unified Diff: ios/chrome/browser/dom_distiller/favicon_web_state_dispatcher_impl.mm

Issue 2529283002: Save favicon during reading list distillation (Closed)
Patch Set: Add constructor comments Created 4 years 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
Index: ios/chrome/browser/dom_distiller/favicon_web_state_dispatcher_impl.mm
diff --git a/ios/chrome/browser/dom_distiller/favicon_web_state_dispatcher_impl.mm b/ios/chrome/browser/dom_distiller/favicon_web_state_dispatcher_impl.mm
new file mode 100644
index 0000000000000000000000000000000000000000..1e58b61a45fd472a5b99a91d6922b81f2ce04e6b
--- /dev/null
+++ b/ios/chrome/browser/dom_distiller/favicon_web_state_dispatcher_impl.mm
@@ -0,0 +1,70 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "ios/chrome/browser/dom_distiller/favicon_web_state_dispatcher_impl.h"
+
+#include "components/favicon/ios/web_favicon_driver.h"
+#include "components/keyed_service/core/service_access_type.h"
+#include "ios/chrome/browser/bookmarks/bookmark_model_factory.h"
+#include "ios/chrome/browser/browser_state/chrome_browser_state.h"
+#include "ios/chrome/browser/favicon/favicon_service_factory.h"
+#include "ios/chrome/browser/history/history_service_factory.h"
+#import "ios/web/public/web_state/web_state.h"
+
+namespace {
+// Default delay to download the favicon when the WebState is handed back.
+const int kDefaultDelayFavicon = 10;
+}
+
+namespace dom_distiller {
+
+FaviconWebStateDispatcherImpl::FaviconWebStateDispatcherImpl(
+ web::BrowserState* browser_state)
+ : FaviconWebStateDispatcherImpl(browser_state, -1) {}
+FaviconWebStateDispatcherImpl::FaviconWebStateDispatcherImpl(
+ web::BrowserState* browser_state,
+ int keep_alive_time)
+ : FaviconWebStateDispatcher(),
+ browser_state_(browser_state),
+ keep_alive_time_(keep_alive_time),
+ weak_ptr_factory_(this) {}
+FaviconWebStateDispatcherImpl::~FaviconWebStateDispatcherImpl() {}
+
+web::WebState* FaviconWebStateDispatcherImpl::RequestWebState() {
+ const web::WebState::CreateParams web_state_create_params(browser_state_);
+ std::unique_ptr<web::WebState> web_state_unique =
+ web::WebState::Create(web_state_create_params);
+ web::WebState* web_state = web_state_unique.get();
+
+ web_states_[web_state] = std::move(web_state_unique);
+
+ ios::ChromeBrowserState* original_browser_state =
+ ios::ChromeBrowserState::FromBrowserState(browser_state_);
+
+ favicon::WebFaviconDriver::CreateForWebState(
+ web_state,
+ ios::FaviconServiceFactory::GetForBrowserState(
+ original_browser_state, ServiceAccessType::EXPLICIT_ACCESS),
+ ios::HistoryServiceFactory::GetForBrowserState(
+ original_browser_state, ServiceAccessType::EXPLICIT_ACCESS),
+ ios::BookmarkModelFactory::GetForBrowserState(original_browser_state));
+
+ return web_state;
+}
+
+void FaviconWebStateDispatcherImpl::ReturnWebState(web::WebState* web_state) {
+ base::WeakPtr<FaviconWebStateDispatcherImpl> weak_this =
+ weak_ptr_factory_.GetWeakPtr();
+ int callback_time =
+ keep_alive_time_ == -1 ? kDefaultDelayFavicon : keep_alive_time_;
+ dispatch_after(
+ dispatch_time(DISPATCH_TIME_NOW, (int64_t)(callback_time * NSEC_PER_SEC)),
+ dispatch_get_main_queue(), ^{
+ FaviconWebStateDispatcherImpl* web_state_dispatcher = weak_this.get();
+ if (web_state_dispatcher)
+ web_state_dispatcher->web_states_.erase(web_state);
+ });
+}
+
+} // namespace dom_distiller

Powered by Google App Engine
This is Rietveld 408576698