Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Webview does not survive a window reload (running out of sources) #120157

Closed
bpasero opened this issue Mar 30, 2021 · 4 comments
Closed

Webview does not survive a window reload (running out of sources) #120157

bpasero opened this issue Mar 30, 2021 · 4 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug webview Webview issues

Comments

@bpasero
Copy link
Member

bpasero commented Mar 30, 2021

Steps to Reproduce:

  1. running out of sources
  2. open a markdown file with Hello World as content
  3. open a markdown preview to the side
  4. reload window

=> 🐛

image

ERR Cannot read property 'resource' of undefined: TypeError: Cannot read property 'resource' of undefined
	at MarkdownPreviewManager.deserializeWebviewPanel (/Users/bpasero/Development/Microsoft/monaco/extensions/markdown-language-features/out/features/previewManager.js:106:49)
	at ExtHostWebviewPanels.$deserializeWebviewPanel (/Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/api/common/extHostWebviewPanels.js:220:30)
	at RPCProtocol._doInvokeHandler (/Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/services/extensions/common/rpcProtocol.js:348:27)
	at RPCProtocol._invokeHandler (/Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/services/extensions/common/rpcProtocol.js:333:45)
	at RPCProtocol._receiveRequest (/Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/services/extensions/common/rpcProtocol.js:260:32)
	at RPCProtocol._receiveOneMessage (/Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/services/extensions/common/rpcProtocol.js:188:26)
	at /Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/services/extensions/common/rpcProtocol.js:69:52
	at Emitter.fire (/Users/bpasero/Development/Microsoft/monaco/out/vs/base/common/event.js:479:38)
	at BufferedEmitter.fire (/Users/bpasero/Development/Microsoft/monaco/out/vs/base/parts/ipc/common/ipc.net.js:375:35)
	at /Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/services/extensions/node/extensionHostProcessSetup.js:146:41
	at Emitter.fire (/Users/bpasero/Development/Microsoft/monaco/out/vs/base/common/event.js:479:38)
	at BufferedEmitter.fire (/Users/bpasero/Development/Microsoft/monaco/out/vs/base/parts/ipc/common/ipc.net.js:375:35)
	at PersistentProtocol._receiveMessage (/Users/bpasero/Development/Microsoft/monaco/out/vs/base/parts/ipc/common/ipc.net.js:654:41)
	at /Users/bpasero/Development/Microsoft/monaco/out/vs/base/parts/ipc/common/ipc.net.js:510:83
	at Emitter.fire (/Users/bpasero/Development/Microsoft/monaco/out/vs/base/common/event.js:479:38)
	at ProtocolReader.acceptChunk (/Users/bpasero/Development/Microsoft/monaco/out/vs/base/parts/ipc/common/ipc.net.js:189:37)
	at /Users/bpasero/Development/Microsoft/monaco/out/vs/base/parts/ipc/common/ipc.net.js:158:61
	at Socket.listener (/Users/bpasero/Development/Microsoft/monaco/out/vs/base/parts/ipc/node/ipc.net.js:33:40)
	at Socket.emit (events.js:315:20)
	at addChunk (_stream_readable.js:295:12)
	at readableAddChunk (_stream_readable.js:271:9)
	at Socket.push (_stream_readable.js:212:10)
	at Pipe.onStreamRead (internal/stream_base_commons.js:186:23)
@bpasero bpasero added bug Issue identified by VS Code Team member as probable bug webview Webview issues labels Mar 30, 2021
@mjbvz
Copy link
Collaborator

mjbvz commented Apr 27, 2021

This happens when the preview previously failed to serialize itself and then tries to restore itself on reload. I have not seen it happen in production. It usually seems caused by having local changes that break some element of webview sterilization logic

Let me know if you see this in insiders or in OSS while working on unrelated code

@mjbvz mjbvz closed this as completed Apr 27, 2021
@bpasero
Copy link
Member Author

bpasero commented Apr 27, 2021

I literally do this:

  • explorer right click on a markdown file -> open preview
  • reload window

It fails to restore.

What is so specific about running out of sources?

@bpasero bpasero reopened this Apr 27, 2021
@mjbvz
Copy link
Collaborator

mjbvz commented Apr 28, 2021

I can't reproduce this:

example.mov

This is running code with ./scripts/code.sh. I've tried both window reloads and killing the process and then restarting

@bpasero
Copy link
Member Author

bpasero commented Apr 28, 2021

So weird and now I cannot reproduce anymore...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug webview Webview issues
Projects
None yet
Development

No branches or pull requests

2 participants