Merge pull request #3961 from Tyriar/canvas_fix2

Remove instantiation service from canvas addon completely

addons/xterm-addon-canvas/src/CanvasAddon.ts

@@ -3,10 +3,10 @@
  * @license MIT
  */
 
-import { ICharSizeService, IRenderService } from 'browser/services/Services';
+import { ICharacterJoinerService, ICharSizeService, ICoreBrowserService, IRenderService } from 'browser/services/Services';
 import { IColorSet } from 'browser/Types';
 import { CanvasRenderer } from './CanvasRenderer';
-import { IBufferService, IInstantiationService, IOptionsService } from 'common/services/Services';
+import { IBufferService, ICoreService, IDecorationService, IOptionsService } from 'common/services/Services';
 import { ITerminalAddon, Terminal } from 'xterm';
 
 export class CanvasAddon implements ITerminalAddon {
@@ -18,15 +18,18 @@ export class CanvasAddon implements ITerminalAddon {
  throw new Error('Cannot activate CanvasAddon before Terminal.open');
  }
  this._terminal = terminal;
- const instantiationService: IInstantiationService = (terminal as any)._core._instantiationService;
  const bufferService: IBufferService = (terminal as any)._core._bufferService;
  const renderService: IRenderService = (terminal as any)._core._renderService;
+ const characterJoinerService: ICharacterJoinerService = (terminal as any)._core._characterJoinerService;
  const charSizeService: ICharSizeService = (terminal as any)._core._charSizeService;
+ const coreService: ICoreService = (terminal as any)._core.coreService;
+ const coreBrowserService: ICoreBrowserService = (terminal as any)._core._coreBrowserService;
+ const decorationService: IDecorationService = (terminal as any)._core._decorationService;
  const optionsService: IOptionsService = (terminal as any)._core.optionsService;
  const colors: IColorSet = (terminal as any)._core._colorManager.colors;
  const screenElement: HTMLElement = (terminal as any)._core.screenElement;
  const linkifier = (terminal as any)._core.linkifier2;
- this._renderer = new CanvasRenderer(colors, screenElement, linkifier, instantiationService, bufferService, charSizeService, optionsService);
+ this._renderer = new CanvasRenderer(colors, screenElement, linkifier, bufferService, charSizeService, optionsService, characterJoinerService, coreService, coreBrowserService, decorationService);
  renderService.setRenderer(this._renderer);
  renderService.onResize(bufferService.cols, bufferService.rows);
  }

addons/xterm-addon-canvas/src/CanvasRenderer.ts

@@ -11,8 +11,8 @@ import { IRenderLayer } from './Types';
 import { LinkRenderLayer } from './LinkRenderLayer';
 import { Disposable } from 'common/Lifecycle';
 import { IColorSet, ILinkifier2 } from 'browser/Types';
-import { ICharSizeService } from 'browser/services/Services';
-import { IBufferService, IOptionsService, IInstantiationService } from 'common/services/Services';
+import { ICharacterJoinerService, ICharSizeService, ICoreBrowserService } from 'browser/services/Services';
+import { IBufferService, IOptionsService, IInstantiationService, IDecorationService, ICoreService } from 'common/services/Services';
 import { removeTerminalFromCache } from './atlas/CharAtlasCache';
 import { EventEmitter, IEvent } from 'common/EventEmitter';
 import { observeDevicePixelDimensions } from 'browser/renderer/DevicePixelObserver';
@@ -34,18 +34,21 @@ export class CanvasRenderer extends Disposable implements IRenderer {
  private _colors: IColorSet,
  private readonly _screenElement: HTMLElement,
  linkifier2: ILinkifier2,
- instantiationService: IInstantiationService,
  private readonly _bufferService: IBufferService,
  private readonly _charSizeService: ICharSizeService,
- private readonly _optionsService: IOptionsService
+ private readonly _optionsService: IOptionsService,
+ characterJoinerService: ICharacterJoinerService,
+ coreService: ICoreService,
+ coreBrowserService: ICoreBrowserService,
+ decorationService: IDecorationService
  ) {
  super();
  const allowTransparency = this._optionsService.rawOptions.allowTransparency;
  this._renderLayers = [
- instantiationService.createInstance(TextRenderLayer, this._screenElement, 0, this._colors, allowTransparency, this._id),
- instantiationService.createInstance(SelectionRenderLayer, this._screenElement, 1, this._colors, this._id),
- instantiationService.createInstance(LinkRenderLayer, this._screenElement, 2, this._colors, this._id, linkifier2),
- instantiationService.createInstance(CursorRenderLayer, this._screenElement, 3, this._colors, this._id, this._onRequestRedraw)
+ new TextRenderLayer(this._screenElement, 0, this._colors, allowTransparency, this._id, this._bufferService, this._optionsService, characterJoinerService, decorationService),
+ new SelectionRenderLayer(this._screenElement, 1, this._colors, this._id, this._bufferService, this._optionsService, decorationService),
+ new LinkRenderLayer(this._screenElement, 2, this._colors, this._id, linkifier2, this._bufferService, this._optionsService, decorationService),
+ new CursorRenderLayer(this._screenElement, 3, this._colors, this._id, this._onRequestRedraw, this._bufferService, this._optionsService, coreService, coreBrowserService, decorationService)
  ];
  this.dimensions = {
  scaledCharWidth: 0,

addons/xterm-addon-canvas/src/CursorRenderLayer.ts

@@ -37,11 +37,11 @@ export class CursorRenderLayer extends BaseRenderLayer {
  colors: IColorSet,
  rendererId: number,
  private _onRequestRedraw: IEventEmitter<IRequestRedrawEvent>,
- @IBufferService bufferService: IBufferService,
- @IOptionsService optionsService: IOptionsService,
- @ICoreService private readonly _coreService: ICoreService,
- @ICoreBrowserService private readonly _coreBrowserService: ICoreBrowserService,
- @IDecorationService decorationService: IDecorationService
+ bufferService: IBufferService,
+ optionsService: IOptionsService,
+ private readonly _coreService: ICoreService,
+ private readonly _coreBrowserService: ICoreBrowserService,
+ decorationService: IDecorationService
  ) {
  super(container, 'cursor', zIndex, true, colors, rendererId, bufferService, optionsService, decorationService);
  this._state = {

addons/xterm-addon-canvas/src/LinkRenderLayer.ts

@@ -19,9 +19,9 @@ export class LinkRenderLayer extends BaseRenderLayer {
  colors: IColorSet,
  rendererId: number,
  linkifier2: ILinkifier2,
- @IBufferService bufferService: IBufferService,
- @IOptionsService optionsService: IOptionsService,
- @IDecorationService decorationService: IDecorationService
+ bufferService: IBufferService,
+ optionsService: IOptionsService,
+ decorationService: IDecorationService
  ) {
  super(container, 'link', zIndex, true, colors, rendererId, bufferService, optionsService, decorationService);
 

addons/xterm-addon-canvas/src/SelectionRenderLayer.ts

@@ -23,9 +23,9 @@ export class SelectionRenderLayer extends BaseRenderLayer {
  zIndex: number,
  colors: IColorSet,
  rendererId: number,
- @IBufferService bufferService: IBufferService,
- @IOptionsService optionsService: IOptionsService,
- @IDecorationService decorationService: IDecorationService
+ bufferService: IBufferService,
+ optionsService: IOptionsService,
+ decorationService: IDecorationService
  ) {
  super(container, 'selection', zIndex, true, colors, rendererId, bufferService, optionsService, decorationService);
  this._clearState();

addons/xterm-addon-canvas/src/TextRenderLayer.ts

@@ -35,10 +35,10 @@ export class TextRenderLayer extends BaseRenderLayer {
  colors: IColorSet,
  alpha: boolean,
  rendererId: number,
- @IBufferService bufferService: IBufferService,
- @IOptionsService optionsService: IOptionsService,
- @ICharacterJoinerService private readonly _characterJoinerService: ICharacterJoinerService,
- @IDecorationService decorationService: IDecorationService
+ bufferService: IBufferService,
+ optionsService: IOptionsService,
+ private readonly _characterJoinerService: ICharacterJoinerService,
+ decorationService: IDecorationService
  ) {
  super(container, 'text', zIndex, alpha, colors, rendererId, bufferService, optionsService, decorationService);
  this._state = new GridCache<CharData>();

addons/xterm-addon-canvas/src/tsconfig.json

@@ -21,7 +21,6 @@
  },
  "strict": true,
  "downlevelIteration": true,
- "experimentalDecorators": true,
  "types": [
  "../../../node_modules/@types/mocha"
  ]

src/browser/Terminal.ts

@@ -81,6 +81,7 @@ export class Terminal extends CoreTerminal implements ITerminal {
  // browser services
  private _decorationService: DecorationService;
  private _charSizeService: ICharSizeService | undefined;
+ private _coreBrowserService: ICoreBrowserService | undefined;
  private _mouseService: IMouseService | undefined;
  private _renderService: IRenderService | undefined;
  private _characterJoinerService: ICharacterJoinerService | undefined;
@@ -494,8 +495,8 @@ export class Terminal extends CoreTerminal implements ITerminal {
  this.register(addDisposableDomListener(this.textarea, 'blur', () => this._onTextAreaBlur()));
  this._helperContainer.appendChild(this.textarea);
 
- const coreBrowserService = this._instantiationService.createInstance(CoreBrowserService, this.textarea);
- this._instantiationService.setService(ICoreBrowserService, coreBrowserService);
+ this._coreBrowserService = this._instantiationService.createInstance(CoreBrowserService, this.textarea);
+ this._instantiationService.setService(ICoreBrowserService, this._coreBrowserService);
 
  this._charSizeService = this._instantiationService.createInstance(CharSizeService, this._document, this._helperContainer);
  this._instantiationService.setService(ICharSizeService, this._charSizeService);