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

Unified Diff: content/child/service_worker/web_service_worker_impl.cc

Issue 2537953003: WebString: makes string16 conversions explicit (part 1: blink, content) (Closed)
Patch Set: fix Created 4 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: content/child/service_worker/web_service_worker_impl.cc
diff --git a/content/child/service_worker/web_service_worker_impl.cc b/content/child/service_worker/web_service_worker_impl.cc
index cdbb0b0462df3a1c52dc3975f70b70b265d5c0e1..947359eb70a9e7f794c2934d61a3ffa0a60af94c 100644
--- a/content/child/service_worker/web_service_worker_impl.cc
+++ b/content/child/service_worker/web_service_worker_impl.cc
@@ -117,10 +117,9 @@ void WebServiceWorkerImpl::postMessage(
base::Bind(&SendPostMessageToWorkerOnMainThread,
base::RetainedRef(thread_safe_sender_),
handle_ref_->handle_id(), provider_impl->provider_id(),
- // We cast WebString to string16 before crossing
+ // We convert WebString to string16 before crossing
// threads for thread-safety.
- static_cast<base::string16>(message),
- url::Origin(source_origin),
+ message.utf16(), url::Origin(source_origin),
base::Passed(base::WrapUnique(channels))));
}
« no previous file with comments | « content/child/service_worker/service_worker_dispatcher.cc ('k') | content/child/web_database_observer_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698