Merge pull request #11078 from wvanlint/opt_in_registration

Add TextEditors to the registry only when opting in
This commit is contained in:
Nathan Sobo
2016-03-11 15:37:07 -07:00
2 changed files with 9 additions and 7 deletions

View File

@@ -548,7 +548,10 @@ class Workspace extends Model
throw error
@project.bufferForPath(filePath, options).then (buffer) =>
@buildTextEditor(_.extend({buffer, largeFileMode}, options))
editor = @buildTextEditor(_.extend({buffer, largeFileMode}, options))
disposable = atom.textEditors.add(editor)
editor.onDidDestroy -> disposable.dispose()
editor
# Public: Returns a {Boolean} that is `true` if `object` is a `TextEditor`.
#
@@ -564,10 +567,7 @@ class Workspace extends Model
@config, @notificationManager, @packageManager, @clipboard, @viewRegistry,
@grammarRegistry, @project, @assert, @applicationDelegate
}, params)
editor = new TextEditor(params)
disposable = atom.textEditors.add(editor)
editor.onDidDestroy -> disposable.dispose()
editor
new TextEditor(params)
# Public: Asynchronously reopens the last-closed item's URI if it hasn't already been
# reopened.