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

Unified Diff: chrome/browser/ui/app_list/search/people/people_provider.cc

Issue 1550053002: Convert Pass()→std::move() in //chrome/browser/ui (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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/ui/app_list/search/people/people_provider.cc
diff --git a/chrome/browser/ui/app_list/search/people/people_provider.cc b/chrome/browser/ui/app_list/search/people/people_provider.cc
index 92d626d1d93cb2b6ec03b64aeb67ac8ba74bbea7..724faf506b96f8fdc3ad53b47487b12469739735 100644
--- a/chrome/browser/ui/app_list/search/people/people_provider.cc
+++ b/chrome/browser/ui/app_list/search/people/people_provider.cc
@@ -5,6 +5,7 @@
#include "chrome/browser/ui/app_list/search/people/people_provider.h"
#include <string>
+#include <utility>
#include "base/bind.h"
#include "base/callback.h"
@@ -155,7 +156,7 @@ void PeopleProvider::StartQuery() {
void PeopleProvider::OnPeopleSearchFetched(
scoped_ptr<base::DictionaryValue> json) {
ProcessPeopleSearchResults(json.get());
- cache_->Put(WebserviceCache::PEOPLE, query_, json.Pass());
+ cache_->Put(WebserviceCache::PEOPLE, query_, std::move(json));
if (!people_search_fetched_callback_.is_null())
people_search_fetched_callback_.Run();
@@ -183,7 +184,7 @@ void PeopleProvider::ProcessPeopleSearchResults(
if (!result)
continue;
- Add(result.Pass());
+ Add(std::move(result));
}
}
@@ -193,10 +194,10 @@ scoped_ptr<SearchResult> PeopleProvider::CreateResult(
scoped_ptr<Person> person = Person::Create(dict);
if (!person)
- return result.Pass();
+ return result;
- result.reset(new PeopleResult(profile_, controller_, person.Pass()));
- return result.Pass();
+ result.reset(new PeopleResult(profile_, controller_, std::move(person)));
+ return result;
}
void PeopleProvider::SetupForTest(

Powered by Google App Engine
This is Rietveld 408576698