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

Unified Diff: chrome/browser/extensions/extension_action.cc

Issue 13949011: Cleanup: Remove unnecessary ".get()" from scoped_ptrs<>. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 8 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
Index: chrome/browser/extensions/extension_action.cc
diff --git a/chrome/browser/extensions/extension_action.cc b/chrome/browser/extensions/extension_action.cc
index 942a84d2df60bc15007254c5ed5e6c2fad059ae9..d8ec44f895d76d3cfba2b66a38a8bd414bef317f 100644
--- a/chrome/browser/extensions/extension_action.cc
+++ b/chrome/browser/extensions/extension_action.cc
@@ -149,12 +149,12 @@ ExtensionAction::IconAnimation::ScopedObserver::ScopedObserver(
Observer* observer)
: icon_animation_(icon_animation),
observer_(observer) {
- if (icon_animation.get())
+ if (icon_animation)
icon_animation->AddObserver(observer);
}
ExtensionAction::IconAnimation::ScopedObserver::~ScopedObserver() {
- if (icon_animation_.get())
+ if (icon_animation_)
icon_animation_->RemoveObserver(observer_);
}
@@ -196,7 +196,7 @@ scoped_ptr<ExtensionAction> ExtensionAction::CopyForTest() const {
copy->icon_animation_ = icon_animation_;
copy->id_ = id_;
- if (default_icon_.get())
+ if (default_icon_)
copy->default_icon_.reset(new ExtensionIconSet(*default_icon_));
return copy.Pass();
@@ -339,7 +339,7 @@ int ExtensionAction::GetIconWidth(int tab_id) const {
return icon.width();
// If there is a default icon, the icon width will be set depending on our
// action type.
- if (default_icon_.get())
+ if (default_icon_)
return GetIconSizeForType(action_type());
// If no icon has been set and there is no default icon, we need favicon
« no previous file with comments | « chrome/browser/extensions/event_listener_map.cc ('k') | chrome/browser/extensions/extension_action_icon_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698