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

Unified Diff: chrome/browser/win/jumplist_updater.cc

Issue 2824773002: Rename ScopedComPtr::get() to ScopedComPtr::Get() (Closed)
Patch Set: Update to 5293966 Created 3 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/win/jumplist_updater.cc
diff --git a/chrome/browser/win/jumplist_updater.cc b/chrome/browser/win/jumplist_updater.cc
index df7c772bee1184316eb7c2ce58f83f25bad9aa72..b4231686842e28d7e8dcc55a1b635f25f5b1a571 100644
--- a/chrome/browser/win/jumplist_updater.cc
+++ b/chrome/browser/win/jumplist_updater.cc
@@ -68,14 +68,14 @@ bool AddShellLink(base::win::ScopedComPtr<IObjectCollection> collection,
return false;
if (!base::win::SetStringValueForPropertyStore(
- property_store.get(),
+ property_store.Get(),
PKEY_Title,
item->title().c_str())) {
return false;
}
// Add this IShellLink object to the given collection.
- return SUCCEEDED(collection->AddObject(link.get()));
+ return SUCCEEDED(collection->AddObject(link.Get()));
}
} // namespace
@@ -122,7 +122,7 @@ bool JumpListUpdater::BeginUpdate() {
SCOPED_UMA_HISTOGRAM_TIMER("WinJumplistUpdater.BeginUpdateDuration");
// This instance is expected to be one-time-use only.
- DCHECK(!destination_list_.get());
+ DCHECK(!destination_list_.Get());
// Check preconditions.
if (!JumpListUpdater::IsEnabled() || app_user_model_id_.empty())
@@ -160,7 +160,7 @@ bool JumpListUpdater::CommitUpdate() {
// TODO(chengx): Remove the UMA histogram after fixing http://crbug.com/40407.
SCOPED_UMA_HISTOGRAM_TIMER("WinJumplistUpdater.CommitUpdateDuration");
- if (!destination_list_.get())
+ if (!destination_list_.Get())
return false;
// Commit this transaction and send the updated JumpList to Windows.
@@ -171,7 +171,7 @@ bool JumpListUpdater::AddTasks(const ShellLinkItemList& link_items) {
// TODO(chengx): Remove the UMA histogram after fixing http://crbug.com/40407.
SCOPED_UMA_HISTOGRAM_TIMER("WinJumplistUpdater.AddTasksDuration");
- if (!destination_list_.get())
+ if (!destination_list_.Get())
return false;
// Retrieve the absolute path to "chrome.exe".
@@ -202,7 +202,7 @@ bool JumpListUpdater::AddTasks(const ShellLinkItemList& link_items) {
if (FAILED(result))
return false;
- return SUCCEEDED(destination_list_->AddUserTasks(object_array.get()));
+ return SUCCEEDED(destination_list_->AddUserTasks(object_array.Get()));
}
bool JumpListUpdater::AddCustomCategory(const std::wstring& category_name,
@@ -211,7 +211,7 @@ bool JumpListUpdater::AddCustomCategory(const std::wstring& category_name,
// TODO(chengx): Remove the UMA histogram after fixing http://crbug.com/40407.
SCOPED_UMA_HISTOGRAM_TIMER("WinJumplistUpdater.AddCustomCategoryDuration");
- if (!destination_list_.get())
+ if (!destination_list_.Get())
return false;
// Retrieve the absolute path to "chrome.exe".
@@ -253,5 +253,5 @@ bool JumpListUpdater::AddCustomCategory(const std::wstring& category_name,
return false;
return SUCCEEDED(destination_list_->AppendCategory(category_name.c_str(),
- object_array.get()));
+ object_array.Get()));
}
« no previous file with comments | « chrome/browser/ui/views/status_icons/status_tray_state_changer_win.cc ('k') | chrome/browser/win/settings_app_monitor.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698