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

Unified Diff: chrome/browser/ui/gtk/browser_actions_toolbar_gtk.cc

Issue 11275088: Remove implicit scoped_refptr operator T* Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 8 years, 2 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/ui/browser.cc ('k') | chrome/browser/ui/toolbar/action_box_button_controller.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/gtk/browser_actions_toolbar_gtk.cc
diff --git a/chrome/browser/ui/gtk/browser_actions_toolbar_gtk.cc b/chrome/browser/ui/gtk/browser_actions_toolbar_gtk.cc
index bcdb5150ce7d812c127ba948b1f1416b0cb0b0f2..e492e26201203d376577a5305e938ebbc87e41ad 100644
--- a/chrome/browser/ui/gtk/browser_actions_toolbar_gtk.cc
+++ b/chrome/browser/ui/gtk/browser_actions_toolbar_gtk.cc
@@ -630,7 +630,7 @@ void BrowserActionsToolbarGtk::CreateAllButtons() {
const extensions::ExtensionList& toolbar_items = model_->toolbar_items();
for (extensions::ExtensionList::const_iterator iter = toolbar_items.begin();
iter != toolbar_items.end(); ++iter) {
- CreateButtonForExtension(*iter, i++);
+ CreateButtonForExtension(*iter.get(), i++);
}
}
@@ -799,7 +799,7 @@ bool BrowserActionsToolbarGtk::IsCommandIdChecked(int command_id) const {
}
bool BrowserActionsToolbarGtk::IsCommandIdEnabled(int command_id) const {
- const Extension* extension = model_->toolbar_items()[command_id];
+ const Extension* extension = model_->toolbar_items()[command_id].get();
return ExtensionActionManager::Get(profile_)->
GetBrowserAction(*extension)->GetIsVisible(GetCurrentTabId());
}
@@ -811,7 +811,7 @@ bool BrowserActionsToolbarGtk::GetAcceleratorForCommandId(
}
void BrowserActionsToolbarGtk::ExecuteCommand(int command_id) {
- const Extension* extension = model_->toolbar_items()[command_id];
+ const Extension* extension = model_->toolbar_items()[command_id].get();
GURL popup_url;
switch (model_->ExecuteBrowserAction(extension, browser(), &popup_url)) {
@@ -1042,7 +1042,7 @@ gboolean BrowserActionsToolbarGtk::OnOverflowButtonPress(
if (profile_->IsOffTheRecord())
model_index = model_->IncognitoIndexToOriginal(i);
- const Extension* extension = model_->toolbar_items()[model_index];
+ const Extension* extension = model_->toolbar_items()[model_index].get();
BrowserActionButton* button = extension_button_map_[extension->id()].get();
overflow_menu_model_->AddItem(model_index, UTF8ToUTF16(extension->name()));
@@ -1082,7 +1082,7 @@ gboolean BrowserActionsToolbarGtk::OnOverflowMenuButtonPress(
if (profile_->IsOffTheRecord())
item_index = model_->IncognitoIndexToOriginal(item_index);
- const Extension* extension = model_->toolbar_items()[item_index];
+ const Extension* extension = model_->toolbar_items()[item_index].get();
ExtensionButtonMap::iterator it = extension_button_map_.find(
extension->id());
if (it == extension_button_map_.end()) {
« no previous file with comments | « chrome/browser/ui/browser.cc ('k') | chrome/browser/ui/toolbar/action_box_button_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698