Skip to content
This repository was archived by the owner on Jun 28, 2022. It is now read-only.

Commit a8f6a6a

Browse files
committed
runat-1128988: Don't pass terminal to addDebugAids
terminal is available on the options object, so it's not needed. Signed-off-by: Joe Walker <jwalker@mozilla.com>
1 parent fc6a032 commit a8f6a6a

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

lib/gcli/test/index.js

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,7 @@ require('./suite');
2828
/**
2929
* Some tricks to make studying the command line state easier
3030
*/
31-
var addDebugAids = exports.addDebugAids = function(options, terminal) {
32-
var requisition = terminal.language.requisition;
33-
31+
var addDebugAids = exports.addDebugAids = function(options) {
3432
window.createDebugCheck = function() {
3533
helpers._createDebugCheck(options).then(function() {
3634
// Don't inline this - chrome console suckage
@@ -40,16 +38,16 @@ var addDebugAids = exports.addDebugAids = function(options, terminal) {
4038

4139
window.summaryJson = function() {
4240
var args = [ 'Requisition: ' ];
43-
var summary = requisition._summaryJson;
41+
var summary = options.terminal.language.requisition._summaryJson;
4442
Object.keys(summary).forEach(function(name) {
4543
args.push(' ' + name + '=');
4644
args.push(summary[name]);
4745
});
4846
console.log.apply(console, args);
4947

5048
console.log('Focus: ' +
51-
'tooltip=', terminal.focusManager._shouldShowTooltip(),
52-
'output=', terminal.focusManager._shouldShowOutput());
49+
'tooltip=', options.terminal.focusManager._shouldShowTooltip(),
50+
'output=', options.terminal.focusManager._shouldShowOutput());
5351
};
5452

5553
document.addEventListener('keyup', function(ev) {
@@ -83,7 +81,7 @@ exports.run = function(options) {
8381
options.isRemote = (options.connection != null);
8482
options.hideExec = true;
8583

86-
addDebugAids(options, options.terminal);
84+
addDebugAids(options);
8785

8886
// phantom-test.js does phantom.exit() on `document.complete = true`
8987
var closeIfPhantomJs = function() {

0 commit comments

Comments
 (0)