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

Commit 59fb94f

Browse files
committed
Merge pull request #10101 from adobe/kai/clean-mb-livedev
Don't subscribe to deprecated event. Clean up log.
2 parents decd8e4 + 5742874 commit 59fb94f

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

src/LiveDevelopment/LiveDevMultiBrowser.js

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,7 @@ define(function (require, exports, module) {
7777
EditorManager = require("editor/EditorManager"),
7878
EventDispatcher = require("utils/EventDispatcher"),
7979
FileUtils = require("file/FileUtils"),
80+
MainViewManager = require("view/MainViewManager"),
8081
PreferencesDialogs = require("preferences/PreferencesDialogs"),
8182
ProjectManager = require("project/ProjectManager"),
8283
Strings = require("strings"),
@@ -658,9 +659,10 @@ define(function (require, exports, module) {
658659

659660
/**
660661
* @private
662+
* MainViewManager.currentFileChange event handler.
661663
* When switching documents, close the current preview and open a new one.
662664
*/
663-
function _onDocumentChange() {
665+
function _onFileChange() {
664666
var doc = DocumentManager.getCurrentDocument();
665667
if (!isActive() || !doc) {
666668
return;
@@ -796,10 +798,13 @@ define(function (require, exports, module) {
796798
* Initialize the LiveDevelopment module.
797799
*/
798800
function init() {
799-
DocumentManager.on("currentDocumentChange", _onDocumentChange)
801+
MainViewManager
802+
.on("currentFileChange", _onFileChange);
803+
DocumentManager
800804
.on("documentSaved", _onDocumentSaved)
801805
.on("dirtyFlagChange", _onDirtyFlagChange);
802-
ProjectManager.on("beforeProjectClose beforeAppClose", close);
806+
ProjectManager
807+
.on("beforeProjectClose beforeAppClose", close);
803808

804809
// Default transport for live connection messages - can be changed
805810
setTransport(NodeSocketTransport);

src/LiveDevelopment/main.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -220,7 +220,6 @@ define(function main(require, exports, module) {
220220
// since UI status are slightly different btw implementations, it also set
221221
// the corresponding style values.
222222
function _setImplementation(multibrowser) {
223-
console.log('set implementation ' + multibrowser);
224223
if (multibrowser) {
225224
// set implemenation
226225
LiveDevImpl = MultiBrowserLiveDev;

0 commit comments

Comments
 (0)