Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Commit b8a5f01

Browse files
committed
Merge branch 'master' into alf_localization_1.14
2 parents 18897af + 8a806ec commit b8a5f01

File tree

2 files changed

+10
-3
lines changed

2 files changed

+10
-3
lines changed

src/brackets.js

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -258,13 +258,20 @@ define(function (require, exports, module) {
258258
}
259259

260260
brackets.app.getRemoteDebuggingPort(function (err, remote_debugging_port){
261-
if (remote_debugging_port && remote_debugging_port > 0) {
262-
var InfoBar = require('widgets/infobar');
261+
var InfoBar = require('widgets/infobar'),
262+
StringUtils = require("utils/StringUtils");
263+
if ((!err) && remote_debugging_port && remote_debugging_port > 0) {
263264
InfoBar.showInfoBar({
264265
type: "warning",
265266
title: `${Strings.REMOTE_DEBUGGING_ENABLED}${remote_debugging_port}`,
266267
description: ""
267268
});
269+
} else if (err) {
270+
InfoBar.showInfoBar({
271+
type: "error",
272+
title: StringUtils.format(Strings.REMOTE_DEBUGGING_PORT_INVALID, err, 1024, 65534),
273+
description: ""
274+
});
268275
}
269276
});
270277
// Use quiet scrollbars if we aren't on Lion. If we're on Lion, only

src/document/DocumentCommandHandlers.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1643,7 +1643,7 @@ define(function (require, exports, module) {
16431643
result.resolve();
16441644
} else {
16451645
brackets.app.getRemoteDebuggingPort(function (err, port){
1646-
if (port && port > 0) {
1646+
if ((!err) && port && port > 0) {
16471647
Inspector.getDebuggableWindows("127.0.0.1", port)
16481648
.fail(result.reject)
16491649
.done(function (response) {

0 commit comments

Comments
 (0)