Merge pull request #3680 from atom/newline-before-code-block

Add newline before example code block

Add newline before example code block

Merge pull request #3663 from preco21/master

Update as upstream

Merge pull request #3676 from atom/callback-race-condition

Make sure V8 handles are deleted on UI thread

Weak map only needs to be deleted for once

Remove itself from the cleanup list when it is destroyed

It is fine to leak a V8 handle on exit

window.id is no longer available when window is closed

Use BuildPrototype to build prototype

This saves the step of manually keeping the global template object,
which is easy to forget then leak.

Do not manually manage native resources

We should rely on the destructor to cleanup everything, instead of
putting them in the Destroy method.

Make sure V8 Function passed to native code are destroyed on UI thread

Update as upstream

Merge remote-tracking branch 'refs/remotes/atom/master'

Merge pull request #3660 from atom/tray-custom-menu

Add `menu` parameter for Tray.popUpContextMenu

Merge pull request #3659 from lukeapage/patch-1

Update debugging-main-process.md

win: Implement menu parameter

mac: Implement menu parameter

Add `menu` parameter for Tray.popUpContextMenu

Update debugging-main-process.md

Deprecating a property with method of same name causes trouble

Close #3511.

Make sure temp file will be cleaned up when base::Move fails

Don't call Move if there is no need to move

spec: Fix failing tests on win32

Merge pull request #3648 from jviotti/jviotti/fix/exec-file-asar-bat

:checkered_flag: Preserve file extension when extracting from asar

Merge pull request #3650 from arturts/patch-5

:memo: [ci skip] Update to match english docs, fix typos

Merge pull request #3649 from arturts/pt-BR-auto-updater

:memo: [ci skip] Add translation to auto-updater, change links

:memo: [ci skip] fix more typos

:memo: [ci skip] fix typos

Change header

:memo: [ci skip] Update to match english docs, fix typos

:memo: [ci skip] Add translation to auto-updater, change links

:checkered_flag: Preserve file extension when extracting from asar

Currently, when calling `copyFileOut`, the original extension from the
file is lost, and a generic `*.tmp` is added instead.

This becomes problematic in the scenario where we use
`child_process.execFile` on a Windows Batch script that lives inside the
`asar` package.

Windows relies on the extension being present in order to interpret the
script accordingly, which results in the following bug because the
operating system doesn't know what do to with this `*.tmp` file:

```
Error: spawn UNKNOWN
```

Steps to reproduce:

1. Create a dummy batch script (test.bat):

```
@echo off
echo "Hello world"
```

2. Create an electron app that attemps to call this script with
`child_process.execFile`:

```js
var child_process = require('child_process');
var path = require('path');

child_process.execFile(path.join(__dirname, 'test.bat'), function(error, stdout) {
    if (error) throw error;
    console.log(stdout);
});
```

3. Package this small application as an asar archive:

```sh
> asar pack mytestapp app.asar
```

4. Execute the application:

```sh
> electron.exe app.asar
```

Merge pull request #3645 from atom/fix-page-title-set

Delay the page-title-updated event to next tick

Deprecate the page-title-set event

We have two names for the same event, page-title-updated wins.

Delay the page-title-set event to next tick

Emit event when title becomes empty

No need to override TitleWasSet in NativeWindow

Merge pull request #3643 from atom/fix-did-fail-load-crash

Delay the did-fail-provisional-load event to next tick

Delay the did-fail-provisional-load event to next tick

Chrome is doing some stuff after the DidFailProvisionalLoad event, if we
call LoadURL at this time crash would happen.

Put common constants in atom_constants

Merge pull request #3641 from atom/process-no-asar

Add process.noAsar to toggle asar support

docs: process.noAsar

spec: process.noAsar

Add process.noAsar to turn off asar support

Merge pull request #3567 from atom/cors

Enable all origins via CORS header for custom schemes

Lint harder

Merge pull request #3638 from arturts/pt-br-translation

:memo: [ci skip] Add translation to app.md

Merge pull request #3636 from arturts/patch-4

:memo: [ci skip] Update to match english docs.

Merge pull request #3635 from arturts/patch-3

:memo: [ci skip] Fix typos

:memo: [ci skip] Add translation to app.md

:memo: [ci skip] Update to match english docs.

:memo: [ci skip] Fix typos

Linting

Add tests to verify behavior

Add CORS header to file jobs

Fix CORS header code to be cleaner

Enable all origins via CORS header for custom schemes

This PR disables CORS for custom schemes, which allows you to serve Font
resources from custom schemes after using registerCustomSchemeAsSecure

Merge pull request #3634 from atom/fix-crash-on-exit

Delete BridgeTaskRunner when main message loop is ready

Delete BridgeTaskRunner when main message loop is ready

Merge pull request #3628 from lovemyliwu/patch-1

:art: add miss charactor

:art: add miss charactor

Update as upstream

Merge pull request #3626 from arturts/patch-2

Update 'Estilo de código' link.

Merge pull request #3625 from arturts/patch-1

Remove 'Google Translator'-like translation

Merge pull request #3622 from atom/wl-docs-skip-ci

Suggest [ci skip] in documentation-only commits

docs: Mention submitting to MAC costs money

Close #3617.

Merge pull request #3618 from preco21/master

Update as upstream

Merge pull request #3616 from leftstick/master

add packaging translation for ZH-CN

Merge pull request #3612 from gunmetal313/translate_to_ru

init russian lang docs

Update 'Estilo de código' link.

'Estilo de código' in 'Desenvolvimento' section now points to the translated coding-style.md

Remove 'Google Translator'-like translation

Merge pull request #3621 from lukewestby/patch-1

fix misspelling in "rotation" string

Wrap [ci skip] in backticks

Suggest [ci skip] in documentation-only commits

fix typo in "rotation" string

Small fixes

Update as upstream

Merge remote-tracking branch 'refs/remotes/atom/master'

add packaging translation for ZH-CN

init russian lang docs

Bump v0.35.2

Merge pull request #3608 from atom/cp-execfile

Add docs and tests for child_process.execFile

Add executable permission in CopyFileOut

spec: Test execFile and execFileSync

docs: Add notes on spawn and exec

Merge pull request #3604 from niedzielski/patch-2

Update docs for webContents.addWorkSpace

Merge pull request #3603 from niedzielski/patch-1

Update docs for globalShortcut.register

Merge pull request #3606 from preco21/master

Update as upstream

Update as upstream

Merge remote-tracking branch 'refs/remotes/atom/master'

Update docs for webContents.addWorkSpace

Specify that webContents.addWorkSpace cannot be called before DevTools
creation and include example.

Fix: #3536

Update docs for globalShortcut.register

Document return value for globalShortcut.register and common failure scenario.

Fix: #3600

Merge pull request #3595 from jviotti/jviotti/fix/3512/execFile-override-exec-permissions

:bug: Fix missing execution permission bit in execFile override

:bug: Fix missing execution permission bit in execFile override

Consider an electron application that uses `execFile` to run a script
that lives within the application code base:

```coffee
child_process = require 'child_process'
child_process.execFile __dirname + '/script.sh', (error) ->
  throw error if error?
```

An application like this will fail when being packaged in an `asar` with
an following error:

```
Error: spawn EACCES
```

Electron overrides certain `fs` functions to make them work within an
`asar` package. In the case of `execFile`, the file to be executed is
extracted from the `asar` package into a temporary file and ran from
there.

The problem is that during the extraction, the original permissions of
the file are lost.

We workaround this by:

1. Extending `asar.stat` to return whether a file is executable or not,
  which is information that's already saved in the `asar` header.

2. Setting execution permissions on the extracted file if the above
  property holds true.

Fixes: https://github.com/atom/electron/issues/3512

Merge remote-tracking branch 'refs/remotes/atom/master'

Fix wrong deprecation wrappers of BrowserWindow

Merge pull request #3594 from atom/public-env

Add ELECTRON_RUN_AS_NODE environment variable

Add ELECTRON_NO_ATTACH_CONSOLE env var

Close #1556.

docs: Add example

docs: Add Environment Variables