summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
Diffstat
-rw-r--r--chrome/browser/extensions/image_loading_tracker.cc40
-rw-r--r--chrome/browser/extensions/image_loading_tracker.h4
-rw-r--r--chrome/browser/views/browser_actions_container.cc6
-rw-r--r--chrome/browser/views/location_bar_view.cc6
4 files changed, 16 insertions, 40 deletions
diff --git a/chrome/browser/extensions/image_loading_tracker.cc b/chrome/browser/extensions/image_loading_tracker.cc
index 211a07b4..e141391 100644
--- a/chrome/browser/extensions/image_loading_tracker.cc
+++ b/chrome/browser/extensions/image_loading_tracker.cc
@@ -29,19 +29,13 @@ class ImageLoadingTracker::LoadImageTask : public Task {
// we use to communicate back to the entity that wants the image after we
// decode it. |path| is the path to load the image from. |index| is an
// identifier for the image that we pass back to the caller.
- // |max_size| is the maximum size for the loaded image. If the image is
- // larger, it will be resized to fit. It is optional.
LoadImageTask(ImageLoadingTracker* tracker,
const ExtensionResource& resource,
- size_t index,
- gfx::Size* max_size)
+ size_t index)
: callback_loop_(MessageLoop::current()),
tracker_(tracker),
resource_(resource),
- index_(index){
- if (max_size)
- max_size_.reset(new gfx::Size(*max_size));
- }
+ index_(index) {}
void ReportBack(SkBitmap* image) {
DCHECK(image);
@@ -71,18 +65,15 @@ class ImageLoadingTracker::LoadImageTask : public Task {
return; // Unable to decode.
}
- if (max_size_.get()) {
- if (decoded->width() > max_size_->width() ||
- decoded->height() > max_size_->height()) {
- // The bitmap is not the correct size, re-sample.
- int new_width = decoded->width();
- int new_height = decoded->height();
- // Calculate what dimensions to use within the constraints (16x16 max).
- calc_favicon_target_size(&new_width, &new_height);
- *decoded = skia::ImageOperations::Resize(
- *decoded, skia::ImageOperations::RESIZE_LANCZOS3,
- new_width, new_height);
- }
+ if (decoded->width() != kFavIconSize || decoded->height() != kFavIconSize) {
+ // The bitmap is not the correct size, re-sample.
+ int new_width = decoded->width();
+ int new_height = decoded->height();
+ // Calculate what dimensions to use within the constraints (16x16 max).
+ calc_favicon_target_size(&new_width, &new_height);
+ *decoded = skia::ImageOperations::Resize(
+ *decoded, skia::ImageOperations::RESIZE_LANCZOS3,
+ new_width, new_height);
}
ReportBack(decoded.release());
@@ -100,20 +91,15 @@ class ImageLoadingTracker::LoadImageTask : public Task {
// The index of the icon being loaded.
size_t index_;
-
- // The max size for the image. If the image is larger than this, it will be
- // scaled down.
- scoped_ptr<gfx::Size> max_size_;
};
////////////////////////////////////////////////////////////////////////////////
// ImageLoadingTracker
-void ImageLoadingTracker::PostLoadImageTask(const ExtensionResource& resource,
- gfx::Size* max_size) {
+void ImageLoadingTracker::PostLoadImageTask(const ExtensionResource& resource) {
MessageLoop* file_loop = g_browser_process->file_thread()->message_loop();
file_loop->PostTask(FROM_HERE, new LoadImageTask(this, resource,
- posted_count_++, max_size));
+ posted_count_++));
}
void ImageLoadingTracker::OnImageLoaded(SkBitmap* image, size_t index) {
diff --git a/chrome/browser/extensions/image_loading_tracker.h b/chrome/browser/extensions/image_loading_tracker.h
index 732e0e2..9732a00 100644
--- a/chrome/browser/extensions/image_loading_tracker.h
+++ b/chrome/browser/extensions/image_loading_tracker.h
@@ -5,7 +5,6 @@
#ifndef CHROME_BROWSER_EXTENSIONS_IMAGE_LOADING_TRACKER_H_
#define CHROME_BROWSER_EXTENSIONS_IMAGE_LOADING_TRACKER_H_
-#include "base/gfx/size.h"
#include "base/ref_counted.h"
class ExtensionResource;
@@ -46,8 +45,7 @@ class ImageLoadingTracker
// Specify image resource to load. This method must be called a number of
// times equal to the |image_count| arugment to the constructor. Calling it
// any more or less than that is an error.
- void PostLoadImageTask(const ExtensionResource& resource,
- gfx::Size* max_size);
+ void PostLoadImageTask(const ExtensionResource& resource);
private:
class LoadImageTask;
diff --git a/chrome/browser/views/browser_actions_container.cc b/chrome/browser/views/browser_actions_container.cc
index 8d5d20a..4a246f2 100644
--- a/chrome/browser/views/browser_actions_container.cc
+++ b/chrome/browser/views/browser_actions_container.cc
@@ -35,9 +35,6 @@ static const int kHorizontalPadding = 4;
// can draw the badge outside the visual bounds of the contianer.
static const int kControlVertOffset = 6;
-// The maximum dimension of the browser action icons.
-static const int kMaxIconSize = 16;
-
////////////////////////////////////////////////////////////////////////////////
// BrowserActionButton
@@ -130,10 +127,9 @@ BrowserActionButton::BrowserActionButton(
const std::vector<std::string>& icon_paths = browser_action->icon_paths();
browser_action_icons_.resize(icon_paths.size());
tracker_ = new ImageLoadingTracker(this, icon_paths.size());
- gfx::Size max_size(kMaxIconSize, kMaxIconSize);
for (std::vector<std::string>::const_iterator iter = icon_paths.begin();
iter != icon_paths.end(); ++iter) {
- tracker_->PostLoadImageTask(extension->GetResource(*iter), &max_size);
+ tracker_->PostLoadImageTask(extension->GetResource(*iter));
}
registrar_.Add(this, NotificationType::EXTENSION_BROWSER_ACTION_UPDATED,
diff --git a/chrome/browser/views/location_bar_view.cc b/chrome/browser/views/location_bar_view.cc
index 31494ba..b7a97a7 100644
--- a/chrome/browser/views/location_bar_view.cc
+++ b/chrome/browser/views/location_bar_view.cc
@@ -59,9 +59,6 @@ static const int kEntryPadding = 3;
// Padding between the entry and the leading/trailing views.
static const int kInnerPadding = 3;
-// Maximum size of the page action icons.
-static const int kMaxPageActionIconSize = 16;
-
static const SkBitmap* kBackground = NULL;
static const SkBitmap* kPopupBackground = NULL;
@@ -1205,10 +1202,9 @@ LocationBarView::PageActionImageView::PageActionImageView(
const std::vector<std::string>& icon_paths = page_action->icon_paths();
page_action_icons_.resize(icon_paths.size());
tracker_ = new ImageLoadingTracker(this, icon_paths.size());
- gfx::Size max_size(kMaxPageActionIconSize, kMaxPageActionIconSize);
for (std::vector<std::string>::const_iterator iter = icon_paths.begin();
iter != icon_paths.end(); ++iter) {
- tracker_->PostLoadImageTask(extension->GetResource(*iter), &max_size);
+ tracker_->PostLoadImageTask(extension->GetResource(*iter));
}
}
generated by cgit v1.1 at 2025-01-22 00:43:06 +0000