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

Unified Diff: chrome/browser/android/popular_sites.cc

Issue 1840033002: Move popular_sites.* files to c/b/a/ntp (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: synced Created 4 years, 9 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 | « chrome/browser/android/popular_sites.h ('k') | chrome/browser/prefs/browser_prefs.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/android/popular_sites.cc
diff --git a/chrome/browser/android/popular_sites.cc b/chrome/browser/android/popular_sites.cc
deleted file mode 100644
index 7549daa8212cd0cf96589c9fd40661daeb3dcc22..0000000000000000000000000000000000000000
--- a/chrome/browser/android/popular_sites.cc
+++ /dev/null
@@ -1,286 +0,0 @@
-// Copyright 2013 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 "chrome/browser/android/popular_sites.h"
-
-#include <stddef.h>
-
-#include "base/bind.h"
-#include "base/command_line.h"
-#include "base/files/file_path.h"
-#include "base/files/file_util.h"
-#include "base/json/json_reader.h"
-#include "base/path_service.h"
-#include "base/strings/string_util.h"
-#include "base/strings/stringprintf.h"
-#include "base/task_runner_util.h"
-#include "base/time/time.h"
-#include "base/values.h"
-#include "chrome/browser/browser_process.h"
-#include "chrome/browser/net/file_downloader.h"
-#include "chrome/browser/profiles/profile.h"
-#include "chrome/browser/search_engines/template_url_service_factory.h"
-#include "chrome/common/chrome_paths.h"
-#include "chrome/common/chrome_switches.h"
-#include "components/google/core/browser/google_util.h"
-#include "components/pref_registry/pref_registry_syncable.h"
-#include "components/prefs/pref_service.h"
-#include "components/search_engines/search_engine_type.h"
-#include "components/search_engines/template_url_prepopulate_data.h"
-#include "components/search_engines/template_url_service.h"
-#include "components/variations/service/variations_service.h"
-#include "content/public/browser/browser_thread.h"
-
-using content::BrowserThread;
-
-namespace {
-
-const char kPopularSitesURLFormat[] = "https://www.gstatic.com/chrome/ntp/%s";
-const char kPopularSitesServerFilenameFormat[] = "suggested_sites_%s_%s.json";
-const char kPopularSitesDefaultCountryCode[] = "DEFAULT";
-const char kPopularSitesDefaultVersion[] = "3";
-const char kPopularSitesLocalFilename[] = "suggested_sites.json";
-const int kPopularSitesRedownloadIntervalHours = 24;
-
-const char kPopularSitesLastDownloadPref[] = "popular_sites_last_download";
-
-// Extract the country from the default search engine if the default search
-// engine is Google.
-std::string GetDefaultSearchEngineCountryCode(Profile* profile) {
- DCHECK(profile);
-
- base::CommandLine* cmd_line = base::CommandLine::ForCurrentProcess();
- if (!cmd_line->HasSwitch(switches::kEnableNTPSearchEngineCountryDetection))
- return std::string();
-
- const TemplateURLService* template_url_service =
- TemplateURLServiceFactory::GetForProfile(profile);
- DCHECK(template_url_service);
-
- const TemplateURL* default_provider =
- template_url_service->GetDefaultSearchProvider();
- // It's possible to not have a default provider in the case that the default
- // search engine is defined by policy.
- if (default_provider) {
- bool is_google_search_engine =
- TemplateURLPrepopulateData::GetEngineType(
- *default_provider, template_url_service->search_terms_data()) ==
- SearchEngineType::SEARCH_ENGINE_GOOGLE;
-
- if (is_google_search_engine) {
- GURL search_url = default_provider->GenerateSearchURL(
- template_url_service->search_terms_data());
- return google_util::GetGoogleCountryCode(search_url);
- }
- }
-
- return std::string();
-}
-
-// Get the country that the experiment is running under
-std::string GetVariationsServiceCountry() {
- DCHECK(g_browser_process);
- variations::VariationsService* variations_service =
- g_browser_process->variations_service();
- if (variations_service)
- return variations_service->GetStoredPermanentCountry();
- return std::string();
-}
-
-// Find out the country code of the user by using the Google country code if
-// Google is the default search engine set. If Google is not the default search
-// engine use the country provided by VariationsService. Fallback to a default
-// if we can't make an educated guess.
-std::string GetCountryCode(Profile* profile) {
- std::string country_code = GetDefaultSearchEngineCountryCode(profile);
-
- if (country_code.empty())
- country_code = GetVariationsServiceCountry();
-
- if (country_code.empty())
- country_code = kPopularSitesDefaultCountryCode;
-
- return base::ToUpperASCII(country_code);
-}
-
-std::string GetPopularSitesServerFilename(
- Profile* profile,
- const std::string& override_country,
- const std::string& override_version) {
- std::string country =
- !override_country.empty() ? override_country : GetCountryCode(profile);
- std::string version = !override_version.empty() ? override_version
- : kPopularSitesDefaultVersion;
- return base::StringPrintf(kPopularSitesServerFilenameFormat,
- country.c_str(), version.c_str());
-}
-
-GURL GetPopularSitesURL(Profile* profile,
- const std::string& override_country,
- const std::string& override_version) {
- return GURL(base::StringPrintf(
- kPopularSitesURLFormat,
- GetPopularSitesServerFilename(profile, override_country, override_version)
- .c_str()));
-}
-
-GURL GetPopularSitesFallbackURL(Profile* profile) {
- return GetPopularSitesURL(profile, kPopularSitesDefaultCountryCode,
- kPopularSitesDefaultVersion);
-}
-
-base::FilePath GetPopularSitesPath() {
- base::FilePath dir;
- PathService::Get(chrome::DIR_USER_DATA, &dir);
- return dir.AppendASCII(kPopularSitesLocalFilename);
-}
-
-scoped_ptr<std::vector<PopularSites::Site>> ReadAndParseJsonFile(
- const base::FilePath& path) {
- std::string json;
- if (!base::ReadFileToString(path, &json)) {
- DLOG(WARNING) << "Failed reading file";
- return nullptr;
- }
-
- scoped_ptr<base::Value> value =
- base::JSONReader::Read(json, base::JSON_ALLOW_TRAILING_COMMAS);
- base::ListValue* list;
- if (!value || !value->GetAsList(&list)) {
- DLOG(WARNING) << "Failed parsing json";
- return nullptr;
- }
-
- scoped_ptr<std::vector<PopularSites::Site>> sites(
- new std::vector<PopularSites::Site>);
- for (size_t i = 0; i < list->GetSize(); i++) {
- base::DictionaryValue* item;
- if (!list->GetDictionary(i, &item))
- continue;
- base::string16 title;
- std::string url;
- if (!item->GetString("title", &title) || !item->GetString("url", &url))
- continue;
- std::string favicon_url;
- item->GetString("favicon_url", &favicon_url);
- std::string thumbnail_url;
- item->GetString("thumbnail_url", &thumbnail_url);
- std::string large_icon_url;
- item->GetString("large_icon_url", &large_icon_url);
-
- sites->push_back(PopularSites::Site(title, GURL(url), GURL(favicon_url),
- GURL(large_icon_url),
- GURL(thumbnail_url)));
- }
-
- return sites;
-}
-
-} // namespace
-
-PopularSites::Site::Site(const base::string16& title,
- const GURL& url,
- const GURL& favicon_url,
- const GURL& large_icon_url,
- const GURL& thumbnail_url)
- : title(title),
- url(url),
- favicon_url(favicon_url),
- large_icon_url(large_icon_url),
- thumbnail_url(thumbnail_url) {}
-
-PopularSites::Site::~Site() {}
-
-PopularSites::PopularSites(Profile* profile,
- const std::string& override_country,
- const std::string& override_version,
- bool force_download,
- const FinishedCallback& callback)
- : PopularSites(profile,
- GetPopularSitesURL(profile, override_country,
- override_version),
- force_download,
- callback) {}
-
-PopularSites::PopularSites(Profile* profile,
- const GURL& url,
- const FinishedCallback& callback)
- : PopularSites(profile, url, true, callback) {}
-
-PopularSites::~PopularSites() {}
-
-// static
-void PopularSites::RegisterProfilePrefs(
- user_prefs::PrefRegistrySyncable* user_prefs) {
- user_prefs->RegisterInt64Pref(kPopularSitesLastDownloadPref, 0);
-}
-
-PopularSites::PopularSites(Profile* profile,
- const GURL& url,
- bool force_download,
- const FinishedCallback& callback)
- : callback_(callback),
- popular_sites_local_path_(GetPopularSitesPath()),
- profile_(profile),
- weak_ptr_factory_(this) {
- const base::Time last_download_time = base::Time::FromInternalValue(
- profile_->GetPrefs()->GetInt64(kPopularSitesLastDownloadPref));
- const base::TimeDelta time_since_last_download =
- base::Time::Now() - last_download_time;
- const base::TimeDelta redownload_interval =
- base::TimeDelta::FromHours(kPopularSitesRedownloadIntervalHours);
- const bool download_time_is_future = base::Time::Now() < last_download_time;
-
- const bool should_redownload_if_exists =
- force_download || download_time_is_future ||
- (time_since_last_download > redownload_interval);
-
- FetchPopularSites(url, should_redownload_if_exists, false /* is_fallback */);
-}
-
-void PopularSites::FetchPopularSites(const GURL& url,
- bool force_download,
- bool is_fallback) {
- downloader_.reset(
- new FileDownloader(url, popular_sites_local_path_, force_download,
- profile_->GetRequestContext(),
- base::Bind(&PopularSites::OnDownloadDone,
- base::Unretained(this), is_fallback)));
-}
-
-void PopularSites::OnDownloadDone(bool is_fallback, bool success) {
- downloader_.reset();
- if (success) {
- profile_->GetPrefs()->SetInt64(kPopularSitesLastDownloadPref,
- base::Time::Now().ToInternalValue());
- ParseSiteList(popular_sites_local_path_);
- } else if (!is_fallback) {
- DLOG(WARNING) << "Download country site list failed";
- // It is fine to force the download as Fallback is only triggered after a
- // failed download.
- FetchPopularSites(GetPopularSitesFallbackURL(profile_),
- true /* force_download */, true /* is_fallback */);
- } else {
- DLOG(WARNING) << "Download fallback site list failed";
- callback_.Run(false);
- }
-}
-
-void PopularSites::ParseSiteList(const base::FilePath& path) {
- base::PostTaskAndReplyWithResult(
- BrowserThread::GetBlockingPool()
- ->GetTaskRunnerWithShutdownBehavior(
- base::SequencedWorkerPool::CONTINUE_ON_SHUTDOWN)
- .get(),
- FROM_HERE, base::Bind(&ReadAndParseJsonFile, path),
- base::Bind(&PopularSites::OnJsonParsed, weak_ptr_factory_.GetWeakPtr()));
-}
-
-void PopularSites::OnJsonParsed(scoped_ptr<std::vector<Site>> sites) {
- if (sites)
- sites_.swap(*sites);
- else
- sites_.clear();
- callback_.Run(!!sites);
-}
« no previous file with comments | « chrome/browser/android/popular_sites.h ('k') | chrome/browser/prefs/browser_prefs.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698