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

Side by Side Diff: chrome/test/data/extensions/api_test/messaging/connect/test.js

Issue 10909256: Always send the full tab object in ExtensionAction click event. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: kalments Created 8 years, 3 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/test/data/extensions/api_test/browser_action/basics/manifest.json ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 JSON.parse = function() { 5 JSON.parse = function() {
6 return "JSON.parse clobbered by extension."; 6 return "JSON.parse clobbered by extension.";
7 }; 7 };
8 8
9 JSON.stringify = function() { 9 JSON.stringify = function() {
10 return "JSON.stringify clobbered by extension."; 10 return "JSON.stringify clobbered by extension.";
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
58 port.onMessage.addListener(function(msg) { 58 port.onMessage.addListener(function(msg) {
59 chrome.test.assertEq(msg.portName, portName); 59 chrome.test.assertEq(msg.portName, portName);
60 port.disconnect(); 60 port.disconnect();
61 chrome.test.succeed(); 61 chrome.test.succeed();
62 }); 62 });
63 }, 63 },
64 64
65 // Tests that postMessage from the tab and its response works. 65 // Tests that postMessage from the tab and its response works.
66 function postMessageFromTab() { 66 function postMessageFromTab() {
67 chrome.extension.onConnect.addListener(function(port) { 67 chrome.extension.onConnect.addListener(function(port) {
68 chrome.test.assertTrue(Boolean(port.sender.tab.url));
69 chrome.test.assertTrue(Boolean(port.sender.tab.title));
70 chrome.test.assertTrue(Boolean(port.sender.tab.id));
68 port.onMessage.addListener(function(msg) { 71 port.onMessage.addListener(function(msg) {
69 chrome.test.assertTrue(msg.testPostMessageFromTab); 72 chrome.test.assertTrue(msg.testPostMessageFromTab);
70 port.postMessage({success: true, portName: port.name}); 73 port.postMessage({success: true, portName: port.name});
71 chrome.test.log("postMessageFromTab: got message from tab"); 74 chrome.test.log("postMessageFromTab: got message from tab");
72 }); 75 });
73 }); 76 });
74 77
75 var port = chrome.tabs.connect(testTab.id); 78 var port = chrome.tabs.connect(testTab.id);
76 port.postMessage({testPostMessageFromTab: true}); 79 port.postMessage({testPostMessageFromTab: true});
77 chrome.test.log("postMessageFromTab: sent first message to tab"); 80 chrome.test.log("postMessageFromTab: sent first message to tab");
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 function disconnectOnClose() { 164 function disconnectOnClose() {
162 var port = chrome.tabs.connect(testTab.id); 165 var port = chrome.tabs.connect(testTab.id);
163 port.postMessage({testDisconnectOnClose: true}); 166 port.postMessage({testDisconnectOnClose: true});
164 port.onDisconnect.addListener(function() { 167 port.onDisconnect.addListener(function() {
165 chrome.test.succeed(); 168 chrome.test.succeed();
166 testTab = null; // the tab is about:blank now. 169 testTab = null; // the tab is about:blank now.
167 }); 170 });
168 }, 171 },
169 ]); 172 ]);
170 }); 173 });
OLDNEW
« no previous file with comments | « chrome/test/data/extensions/api_test/browser_action/basics/manifest.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698