| Index: chrome/browser/favicon/favicon_service.cc
|
| diff --git a/chrome/browser/favicon/favicon_service.cc b/chrome/browser/favicon/favicon_service.cc
|
| index e2374e7cf2342e6e630b511c10cd98c4859092ae..b39cac7ba38aa19f79c6ba0a296585e8f2697ed6 100644
|
| --- a/chrome/browser/favicon/favicon_service.cc
|
| +++ b/chrome/browser/favicon/favicon_service.cc
|
| @@ -92,9 +92,8 @@ base::CancelableTaskTracker::TaskId FaviconService::GetFaviconImage(
|
| return history_service_->GetFavicons(
|
| icon_urls, icon_type, desired_size_in_dip,
|
| FaviconUtil::GetFaviconScaleFactors(), callback_runner, tracker);
|
| - } else {
|
| - return RunWithEmptyResultAsync(callback_runner, tracker);
|
| }
|
| + return RunWithEmptyResultAsync(callback_runner, tracker);
|
| }
|
|
|
| base::CancelableTaskTracker::TaskId FaviconService::GetRawFavicon(
|
| @@ -118,9 +117,8 @@ base::CancelableTaskTracker::TaskId FaviconService::GetRawFavicon(
|
| return history_service_->GetFavicons(
|
| icon_urls, icon_type, desired_size_in_dip, desired_scale_factors,
|
| callback_runner, tracker);
|
| - } else {
|
| - return RunWithEmptyResultAsync(callback_runner, tracker);
|
| }
|
| + return RunWithEmptyResultAsync(callback_runner, tracker);
|
| }
|
|
|
| base::CancelableTaskTracker::TaskId FaviconService::GetFavicon(
|
| @@ -135,9 +133,8 @@ base::CancelableTaskTracker::TaskId FaviconService::GetFavicon(
|
| return history_service_->GetFavicons(
|
| icon_urls, icon_type, desired_size_in_dip,
|
| FaviconUtil::GetFaviconScaleFactors(), callback, tracker);
|
| - } else {
|
| - return RunWithEmptyResultAsync(callback, tracker);
|
| }
|
| + return RunWithEmptyResultAsync(callback, tracker);
|
| }
|
|
|
| base::CancelableTaskTracker::TaskId
|
| @@ -152,9 +149,8 @@ FaviconService::UpdateFaviconMappingsAndFetch(
|
| return history_service_->UpdateFaviconMappingsAndFetch(
|
| page_url, icon_urls, icon_types, desired_size_in_dip,
|
| FaviconUtil::GetFaviconScaleFactors(), callback, tracker);
|
| - } else {
|
| - return RunWithEmptyResultAsync(callback, tracker);
|
| }
|
| + return RunWithEmptyResultAsync(callback, tracker);
|
| }
|
|
|
| base::CancelableTaskTracker::TaskId FaviconService::GetFaviconImageForURL(
|
| @@ -205,7 +201,8 @@ base::CancelableTaskTracker::TaskId FaviconService::GetLargestRawFaviconForURL(
|
| scale_factor.push_back(ui::SCALE_FACTOR_100P);
|
| return GetFaviconForChromeURL(profile, page_url, scale_factor,
|
| favicon_results_callback, tracker);
|
| - } else if (history_service_) {
|
| + }
|
| + if (history_service_) {
|
| return history_service_->GetLargestFaviconForURL(page_url, icon_types,
|
| minimum_size_in_pixels, callback, tracker);
|
| }
|
| @@ -239,9 +236,8 @@ base::CancelableTaskTracker::TaskId FaviconService::GetLargestRawFaviconForID(
|
| return history_service_->GetFaviconForID(
|
| favicon_id, desired_size_in_dip, desired_scale_factor,
|
| callback_runner, tracker);
|
| - } else {
|
| - return RunWithEmptyResultAsync(callback_runner, tracker);
|
| }
|
| + return RunWithEmptyResultAsync(callback_runner, tracker);
|
| }
|
|
|
| void FaviconService::SetFaviconOutOfDateForPage(const GURL& page_url) {
|
| @@ -329,7 +325,8 @@ base::CancelableTaskTracker::TaskId FaviconService::GetFaviconForURLImpl(
|
| params.page_url.SchemeIs(extensions::kExtensionScheme)) {
|
| return GetFaviconForChromeURL(profile_, params.page_url,
|
| desired_scale_factors, callback, tracker);
|
| - } else if (history_service_) {
|
| + }
|
| + if (history_service_) {
|
| return history_service_->GetFaviconsForURL(params.page_url,
|
| params.icon_types,
|
| params.desired_size_in_dip,
|
| @@ -380,9 +377,10 @@ void FaviconService::RunFaviconRawCallbackWithBitmapResults(
|
| }
|
|
|
| // If history bitmap is already desired pixel size, return early.
|
| - float desired_scale = ui::GetImageScale(desired_scale_factor);
|
| + float desired_scale = ui::GetScaleForScaleFactor(desired_scale_factor);
|
| int desired_edge_width_in_pixel = static_cast<int>(
|
| desired_size_in_dip * desired_scale + 0.5f);
|
| +
|
| gfx::Size desired_size_in_pixel(desired_edge_width_in_pixel,
|
| desired_edge_width_in_pixel);
|
| if (bitmap_result.pixel_size == desired_size_in_pixel) {
|
|
|