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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/extensions/api/browsing_data/browsing_data_api.cc

Issue 1991083002: Remove ExtensionFunction::SetResult(T*) overload. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: IWYU Created 4 years, 7 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/extensions/api/braille_display_private/braille_display_private_api.cc ('k') | chrome/browser/extensions/api/cloud_print_private/cloud_print_private_api.cc » ('j') | chrome/browser/extensions/api/content_settings/content_settings_api.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/api/browsing_data/browsing_data_api.cc
diff --git a/chrome/browser/extensions/api/browsing_data/browsing_data_api.cc b/chrome/browser/extensions/api/browsing_data/browsing_data_api.cc
index 30e7b61e6842376d082c2d74339780bf267150ce..0e329e57d764d6c73249465d53771d95dbe0a68c 100644
--- a/chrome/browser/extensions/api/browsing_data/browsing_data_api.cc
+++ b/chrome/browser/extensions/api/browsing_data/browsing_data_api.cc
@@ -9,6 +9,7 @@
#include "chrome/browser/extensions/api/browsing_data/browsing_data_api.h"
#include <string>
+#include <utility>
#include "base/strings/stringprintf.h"
#include "base/values.h"
@@ -218,7 +219,7 @@ bool BrowsingDataSettingsFunction::RunSync() {
selected.release());
result->Set(extension_browsing_data_api_constants::kDataRemovalPermittedKey,
permitted.release());
- SetResult(result.release());
+ SetResult(std::move(result));
return true;
}
« no previous file with comments | « chrome/browser/extensions/api/braille_display_private/braille_display_private_api.cc ('k') | chrome/browser/extensions/api/cloud_print_private/cloud_print_private_api.cc » ('j') | chrome/browser/extensions/api/content_settings/content_settings_api.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698