Update to electron 1.6.x

.travis.yml

@@ -26,8 +26,8 @@ before_install:
  - git submodule update --init --recursive
  - git clone --depth 1 https://github.com/creationix/nvm.git ./.nvm
  - source ./.nvm/nvm.sh
- - nvm install 6.6.0
- - nvm use 6.6.0
+ - nvm install 7.4.0
+ - nvm use 7.4.0
  - npm config set python `which python`
  - npm install -g gulp
  - if [ $TRAVIS_OS_NAME == "linux" ]; then

OSSREADME.json

@@ -68,7 +68,7 @@
 },
 {
  "name": "chromium",
- "version": "53.0.2785.143",
+ "version": "56.0.2924.87",
  "repositoryURL": "http://www.chromium.org/Home",
  "licenseDetail": [
  "BSD License",
@@ -104,14 +104,14 @@
 },
 {
  "name": "libchromiumcontent",
- "version": "53.0.2785.143",
+ "version": "56.0.2924.87",
  "license": "MIT",
  "repositoryURL": "https://github.com/electron/libchromiumcontent",
  "isProd": true
 },
 {
  "name": "nodejs",
- "version": "6.5.0",
+ "version": "7.4.0",
  "repositoryURL": "https://github.com/nodejs/node",
  "isProd": true
 },

appveyor.yml

@@ -3,7 +3,7 @@ environment:
  VSCODE_BUILD_VERBOSE: true
 
 install:
- - ps: Install-Product node 6.6.0 x64
+ - ps: Install-Product node 7.4.0 x64
  - npm install -g npm --silent
  - npm install -g gulp mocha --silent
 

package.json

@@ -1,7 +1,7 @@
 {
  "name": "code-oss-dev",
  "version": "1.12.0",
- "electronVersion": "1.4.6",
+ "electronVersion": "1.6.5",
  "distro": "fbef8e6a73f122b5c8b0034e2c1549e00a1815c3",
  "author": {
  "name": "Microsoft Corporation"

src/typings/electron.d.ts

@@ -1784,6 +1784,11 @@ declare namespace Electron {
  * Settings of web page’s features.
  */
  webPreferences?: WebPreferences;
+ /**
+ * Tab group name, allows opening the window as a native tab on macOS 10.12+.
+ * Windows with the same tabbing identifier will be grouped together.
+ */
+ tabbingIdentifier?: string;
  }
 
  type BrowserWindowType = BrowserWindowTypeLinux | BrowserWindowTypeMac | BrowserWindowTypeWindows;

src/vs/base/browser/dom.ts

@@ -746,6 +746,7 @@ export function isHTMLElement(o: any): o is HTMLElement {
 export const EventType = {
  // Mouse
  CLICK: 'click',
+ AUXCLICK: 'auxclick', // >= Chrome 56
  DBLCLICK: 'dblclick',
  MOUSE_UP: 'mouseup',
  MOUSE_DOWN: 'mousedown',

src/vs/base/parts/tree/browser/treeView.ts

@@ -488,6 +488,7 @@ export class TreeView extends HeightMap {
  this.viewListeners.push(DOM.addDisposableListener(this.domNode, 'mousedown', (e) => this.onMouseDown(e)));
  this.viewListeners.push(DOM.addDisposableListener(this.domNode, 'mouseup', (e) => this.onMouseUp(e)));
  this.viewListeners.push(DOM.addDisposableListener(this.wrapper, 'click', (e) => this.onClick(e)));
+ this.viewListeners.push(DOM.addDisposableListener(this.wrapper, 'auxclick', (e) => this.onClick(e))); // >= Chrome 56
  this.viewListeners.push(DOM.addDisposableListener(this.domNode, 'contextmenu', (e) => this.onContextMenu(e)));
  this.viewListeners.push(DOM.addDisposableListener(this.wrapper, Touch.EventType.Tap, (e) => this.onTap(e)));
  this.viewListeners.push(DOM.addDisposableListener(this.wrapper, Touch.EventType.Change, (e) => this.onTouchChange(e)));

src/vs/code/electron-main/window.ts

@@ -35,7 +35,6 @@ export interface IWindowCreationOptions {
  state: IWindowState;
  extensionDevelopmentPath?: string;
  isExtensionTestHost?: boolean;
- titleBarStyle?: 'native' | 'custom';
 }
 
 export enum WindowMode {
@@ -190,22 +189,35 @@ export class VSCodeWindow {
  show: !isFullscreenOrMaximized,
  title: product.nameLong,
  webPreferences: {
- 'backgroundThrottling': false // by default if Code is in the background, intervals and timeouts get throttled
+ 'backgroundThrottling': false, // by default if Code is in the background, intervals and timeouts get throttled,
+ disableBlinkFeatures: 'Auxclick' // disable auxclick events (see https://developers.google.com/web/updates/2016/10/auxclick)
  }
  };
 
  if (platform.isLinux) {
  options.icon = path.join(this.environmentService.appRoot, 'resources/linux/code.png'); // Windows and Mac are better off using the embedded icon(s)
  }
 
+ const windowConfig = this.configurationService.getConfiguration<IWindowSettings>('window');
+
+ let useNativeTabs = false;
+ if (windowConfig && windowConfig.nativeTabs) {
+ options.tabbingIdentifier = product.nameShort; // this opts in to sierra tabs
+ useNativeTabs = true;
+ }
+
  let useCustomTitleStyle = false;
- if (platform.isMacintosh && (!this.options.titleBarStyle || this.options.titleBarStyle === 'custom')) {
+ if (platform.isMacintosh && (!windowConfig || !windowConfig.titleBarStyle || windowConfig.titleBarStyle === 'custom')) {
  const isDev = !this.environmentService.isBuilt || !!config.extensionDevelopmentPath;
  if (!isDev) {
  useCustomTitleStyle = true; // not enabled when developing due to https://github.com/electron/electron/issues/3647
  }
  }
 
+ if (useNativeTabs) {
+ useCustomTitleStyle = false; // native tabs on sierra do not work with custom title style
+ }
+
  if (useCustomTitleStyle) {
  options.titleBarStyle = 'hidden';
  this.hiddenTitleBarStyle = true;

src/vs/code/electron-main/windows.ts

@@ -851,8 +851,7 @@ export class WindowsManager implements IWindowsMainService {
  vscodeWindow = new VSCodeWindow({
  state,
  extensionDevelopmentPath: configuration.extensionDevelopmentPath,
- isExtensionTestHost: !!configuration.extensionTestsPath,
- titleBarStyle: windowConfig ? windowConfig.titleBarStyle : void 0
+ isExtensionTestHost: !!configuration.extensionTestsPath
  },
  this.logService,
  this.environmentService,

src/vs/platform/windows/common/windows.ts

@@ -101,4 +101,5 @@ export interface IWindowSettings {
  autoDetectHighContrast: boolean;
  menuBarVisibility: MenuBarVisibility;
  newWindowDimensions: 'default' | 'inherit' | 'maximized' | 'fullscreen';
+ nativeTabs: boolean;
 }

src/vs/workbench/browser/parts/editor/noTabsTitleControl.ts

@@ -34,6 +34,7 @@ export class NoTabsTitleControl extends TitleControl {
 
  // Detect mouse click
  this.toUnbind.push(DOM.addDisposableListener(this.titleContainer, DOM.EventType.CLICK, (e: MouseEvent) => this.onTitleClick(e)));
+ this.toUnbind.push(DOM.addDisposableListener(this.titleContainer, DOM.EventType.AUXCLICK, (e: MouseEvent) => this.onTitleClick(e)));
 
  // Editor Label
  this.editorLabel = this.instantiationService.createInstance(EditorLabel, this.titleContainer, void 0);

src/vs/workbench/electron-browser/main.contribution.ts

@@ -8,6 +8,7 @@
 import { Registry } from 'vs/platform/platform';
 import nls = require('vs/nls');
 import product from 'vs/platform/node/product';
+import * as os from 'os';
 import { SyncActionDescriptor } from 'vs/platform/actions/common/actions';
 import { IConfigurationRegistry, Extensions as ConfigurationExtensions } from 'vs/platform/configuration/common/configurationRegistry';
 import { IWorkbenchActionRegistry, Extensions } from 'vs/workbench/common/actionRegistry';
@@ -269,6 +270,15 @@ if (isMacintosh) {
  'default': 'custom',
  'description': nls.localize('titleBarStyle', "Adjust the appearance of the window title bar. Changes require a full restart to apply.")
  };
+
+ // macOS Sierra (10.12.x = darwin 16.x) only
+ if (os.release().indexOf('16.') === 0) {
+ properties['window.nativeTabs'] = {
+ 'type': 'boolean',
+ 'default': false,
+ 'description': nls.localize('window.nativeTabs', "Enables macOS Sierra window tabs. Note that changes require a full restart to apply and that native tabs will disable a custom title bar style if configured.")
+ };
+ }
 }
 
 configurationRegistry.registerConfiguration({

src/vs/workbench/electron-browser/workbench.ts

@@ -673,11 +673,16 @@ export class Workbench implements IPartService {
  }
 
  const windowConfig = this.configurationService.getConfiguration<IWindowConfiguration>();
+ if (windowConfig && windowConfig.window) {
+ const useNativeTabs = windowConfig.window.nativeTabs;
+ if (useNativeTabs) {
+ return null; // native tabs on sierra do not work with custom title style
+ }
 
- const style = windowConfig && windowConfig.window && windowConfig.window.titleBarStyle;
-
- if (style === 'custom') {
- return style;
+  const style = windowConfig.window.titleBarStyle;
+ if (style === 'custom') {
+  return style;
+ }
  }
 
  return null;

src/vs/workbench/parts/html/browser/webview.ts

@@ -60,6 +60,9 @@ export default class Webview {
  this._webview.style.opacity = '0';
  this._webview.autoSize = 'on';
 
+ // disable auxclick events (see https://developers.google.com/web/updates/2016/10/auxclick)
+ this._webview.setAttribute('disableblinkfeatures', 'Auxclick');
+
  this._webview.preload = require.toUrl('./webview-pre.js');
  this._webview.src = require.toUrl('./webview.html');
 

src/vs/workbench/parts/output/test/bufferedContent.test.ts

@@ -35,7 +35,8 @@ suite('Workbench - Output Buffered Content', () => {
  assert.equal(bufferedContent.getDelta(secondDelta).value, 'fourthfifth');
  });
 
- test('Buffered Content - Lots of Output', () => {
+ test('Buffered Content - Lots of Output', function () {
+ this.timeout(10000);
  const bufferedContent = new BufferedContent();
  bufferedContent.append('first line');
  const firstDelta = bufferedContent.getDelta();

src/vs/workbench/services/textfile/test/textFileEditorModel.test.ts

@@ -244,7 +244,7 @@ suite('Files - TextFileEditorModel', () => {
  model.load().done(() => {
  model.textEditorModel.setValue('foo');
 
- return TPromise.timeout(50).then(() => {
+ return TPromise.timeout(100).then(() => {
  assert.ok(!model.isDirty());
  assert.equal(eventCounter, 2);