Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
xxadev
vscode
提交
744e89f0
V
vscode
项目概览
xxadev
/
vscode
与 Fork 源项目一致
从无法访问的项目Fork
通知
2
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
V
vscode
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
提交
744e89f0
编写于
9月 05, 2018
作者:
A
Alex Dima
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
Fixes #57820: Switch to using native Promises
上级
6272a7fe
变更
1
隐藏空白更改
内联
并排
Showing
1 changed file
with
62 addition
and
4 deletion
+62
-4
src/vs/workbench/services/extensions/electron-browser/extensionHost.ts
...nch/services/extensions/electron-browser/extensionHost.ts
+62
-4
未找到文件。
src/vs/workbench/services/extensions/electron-browser/extensionHost.ts
浏览文件 @
744e89f0
...
...
@@ -41,6 +41,7 @@ import { isMessageOfType, MessageType, createMessageOfType } from 'vs/workbench/
import
{
ILabelService
}
from
'
vs/platform/label/common/label
'
;
import
{
URI
}
from
'
vs/base/common/uri
'
;
import
{
Schemas
}
from
'
vs/base/common/network
'
;
import
{
onUnexpectedError
}
from
'
vs/base/common/errors
'
;
export
interface
IExtensionHostStarter
{
readonly
onCrashed
:
Event
<
[
number
,
string
]
>
;
...
...
@@ -316,9 +317,9 @@ export class ExtensionHostProcessWorker implements IExtensionHostStarter {
});
}
private
_tryExtHostHandshake
():
T
Promise
<
IMessagePassingProtocol
>
{
private
_tryExtHostHandshake
():
Promise
<
IMessagePassingProtocol
>
{
return
new
T
Promise
<
IMessagePassingProtocol
>
((
resolve
,
reject
)
=>
{
return
new
Promise
<
IMessagePassingProtocol
>
((
resolve
,
reject
)
=>
{
// Wait for the extension host to connect to our named pipe
// and wrap the socket in the message passing protocol
...
...
@@ -340,7 +341,7 @@ export class ExtensionHostProcessWorker implements IExtensionHostStarter {
// 1) wait for the incoming `ready` event and send the initialization data.
// 2) wait for the incoming `initialized` event.
return
new
T
Promise
<
IMessagePassingProtocol
>
((
resolve
,
reject
)
=>
{
return
new
Promise
<
IMessagePassingProtocol
>
((
resolve
,
reject
)
=>
{
let
handle
=
setTimeout
(()
=>
{
reject
(
'
timeout
'
);
...
...
@@ -363,7 +364,10 @@ export class ExtensionHostProcessWorker implements IExtensionHostStarter {
disposable
.
dispose
();
// release this promise
resolve
(
protocol
);
// using a buffered message protocol here because between now
// and the first time a `then` executes some messages might be lost
// unless we immediately register a listener for `onMessage`.
resolve
(
new
BufferedMessagePassingProtocol
(
protocol
));
return
;
}
...
...
@@ -530,3 +534,57 @@ export class ExtensionHostProcessWorker implements IExtensionHostStarter {
}
}
}
/**
* Will ensure no messages are lost from creation time until the first user of onMessage comes in.
*/
class
BufferedMessagePassingProtocol
implements
IMessagePassingProtocol
{
private
readonly
_actual
:
IMessagePassingProtocol
;
private
_bufferedMessagesListener
:
IDisposable
;
private
_bufferedMessages
:
Buffer
[];
constructor
(
actual
:
IMessagePassingProtocol
)
{
this
.
_actual
=
actual
;
this
.
_bufferedMessages
=
[];
this
.
_bufferedMessagesListener
=
this
.
_actual
.
onMessage
((
buff
)
=>
this
.
_bufferedMessages
.
push
(
buff
));
}
public
send
(
buffer
:
Buffer
):
void
{
this
.
_actual
.
send
(
buffer
);
}
public
onMessage
(
listener
:
(
e
:
Buffer
)
=>
any
,
thisArgs
?:
any
,
disposables
?:
IDisposable
[]):
IDisposable
{
if
(
!
this
.
_bufferedMessages
)
{
// second caller gets nothing
return
this
.
_actual
.
onMessage
(
listener
,
thisArgs
,
disposables
);
}
// prepare result
const
result
=
this
.
_actual
.
onMessage
(
listener
,
thisArgs
,
disposables
);
// stop listening to buffered messages
this
.
_bufferedMessagesListener
.
dispose
();
// capture buffered messages
const
bufferedMessages
=
this
.
_bufferedMessages
;
this
.
_bufferedMessages
=
null
;
// it is important to deliver these messages after this call, but before
// other messages have a chance to be received (to guarantee in order delivery)
// that's why we're using here nextTick and not other types of timeouts
process
.
nextTick
(()
=>
{
// deliver buffered messages
while
(
bufferedMessages
.
length
>
0
)
{
const
msg
=
bufferedMessages
.
shift
();
try
{
listener
.
call
(
thisArgs
,
msg
);
}
catch
(
e
)
{
onUnexpectedError
(
e
);
}
}
});
return
result
;
}
}
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录