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

Unified Diff: trunk/src/chrome/browser/extensions/api/messaging/message_service.cc

Issue 16336011: Revert 203489 "Replace JSON (de)serialization of extension messa..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 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
Index: trunk/src/chrome/browser/extensions/api/messaging/message_service.cc
===================================================================
--- trunk/src/chrome/browser/extensions/api/messaging/message_service.cc (revision 203729)
+++ trunk/src/chrome/browser/extensions/api/messaging/message_service.cc (working copy)
@@ -6,7 +6,6 @@
#include "base/atomic_sequence_num.h"
#include "base/bind.h"
-#include "base/bind_helpers.h"
#include "base/callback.h"
#include "base/json/json_writer.h"
#include "base/lazy_instance.h"
@@ -430,7 +429,7 @@
}
void MessageService::PostMessage(
- int source_port_id, scoped_ptr<base::ListValue> message) {
+ int source_port_id, const std::string& message) {
int channel_id = GET_CHANNEL_ID(source_port_id);
MessageChannelMap::iterator iter = channels_.find(channel_id);
if (iter == channels_.end()) {
@@ -441,9 +440,7 @@
lazy_background_task_queue_->AddPendingTask(
pending->second.first, pending->second.second,
base::Bind(&MessageService::PendingPostMessage,
- weak_factory_.GetWeakPtr(),
- source_port_id,
- base::Passed(&message)));
+ weak_factory_.GetWeakPtr(), source_port_id, message));
}
return;
}
@@ -453,13 +450,12 @@
MessagePort* port = IS_OPENER_PORT_ID(dest_port_id) ?
iter->second->opener.get() : iter->second->receiver.get();
- port->DispatchOnMessage(message.Pass(), dest_port_id);
+ port->DispatchOnMessage(message, dest_port_id);
}
-void MessageService::PostMessageFromNativeProcess(
- int port_id,
- scoped_ptr<base::ListValue> message) {
- PostMessage(port_id, message.Pass());
+void MessageService::PostMessageFromNativeProcess(int port_id,
+ const std::string& message) {
+ PostMessage(port_id, message);
}
void MessageService::Observe(int type,

Powered by Google App Engine
This is Rietveld 408576698