Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: ext/content.js

Issue 5646124035604480: Issue 154 - Added UI for devtools panel on Chrome (Closed)
Patch Set: Added new request types Created Jan. 28, 2016, 7:37 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ext/background.js ('k') | ext/devtools.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ext/content.js
===================================================================
--- a/ext/content.js
+++ b/ext/content.js
@@ -45,23 +45,28 @@
window.addEventListener("message", loadHandler, false);
global.ext.backgroundPage = {
+ _sendRawMessage: function(message)
+ {
+ if (messageQueue)
+ messageQueue.push(message);
+ else
+ backgroundFrame.contentWindow.postMessage(message, "*");
+ },
sendMessage: function(message, responseCallback)
{
- var rawMessage = {
+ var messageId = ++maxMessageId;
+
+ this._sendRawMessage({
type: "message",
- messageId: ++maxMessageId,
+ messageId: messageId,
payload: message
- };
- if (messageQueue)
- messageQueue.push(rawMessage);
- else
- backgroundFrame.contentWindow.postMessage(rawMessage, "*");
+ });
if (responseCallback)
{
var callbackWrapper = function(event)
{
- if (event.data.type == "response" && event.data.messageId == rawMessage.messageId)
+ if (event.data.type == "response" && event.data.messageId == messageId)
{
window.removeEventListener("message", callbackWrapper, false);
responseCallback(event.data.payload);
« no previous file with comments | « ext/background.js ('k') | ext/devtools.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld