Index: chrome/browser/extensions/api/streams_private/streams_private_api.cc |
diff --git a/chrome/browser/extensions/api/streams_private/streams_private_api.cc b/chrome/browser/extensions/api/streams_private/streams_private_api.cc |
index dc12dac808f754a699567fba4e96c960674587f7..20dc1afbacf1df8f20930fc7b5259fbfb8f59914 100644 |
--- a/chrome/browser/extensions/api/streams_private/streams_private_api.cc |
+++ b/chrome/browser/extensions/api/streams_private/streams_private_api.cc |
@@ -39,7 +39,8 @@ void CreateResponseHeadersDictionary(const net::HttpResponseHeaders* headers, |
if (result->Get(header_name, &existing_value)) { |
base::StringValue* existing_string_value = |
static_cast<base::StringValue*>(existing_value); |
- existing_string_value->GetString()->append(", ").append(header_value); |
+ *existing_string_value = base::StringValue( |
+ existing_string_value->GetString() + ", " + std::move(header_value)); |
brettw
2017/01/31 22:22:24
This use of std::move seems not to be helpful to m
jdoerrie
2017/02/01 12:36:26
C++11 introduced overloads for operator+ that take
|
} else { |
result->SetString(header_name, header_value); |
} |