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.

Sign up for GitHub

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

Jump to bottom

Doesn't find PresentationSource hosted in CustomTaskPane (ElementHost) in Office VSTO Add-in #116

Closed
HamletHakobyan opened this issue Dec 21, 2018 · 9 comments
Closed

Doesn't find PresentationSource hosted in CustomTaskPane (ElementHost) in Office VSTO Add-in #116

HamletHakobyan opened this issue Dec 21, 2018 · 9 comments
Assignees
@batzen
Labels
enhancement
Milestone
2.11

Comments

@HamletHakobyan
Copy link

Hi,

Snoop can't find PresentationSource that contains WPF control. I did some investigation and it is not the case which was related to Wpf controls hosted in ElementHost of WinForms window.
The issue arises by specificity of Office VSTO Add-in which loads in its own appdomain.
PresentationSource.CurrentSources can't enumerate sources from another appdomain which is caused the issue.
Attached minimal project for repro.
SnoopTestAddIn.zip
@HamletHakobyan
Copy link
Author

Hi guys, any news?

@batzen
Copy link
Collaborator

batzen commented Jan 18, 2019

I've got a"working"version on my machine. I just need time to finish it and i am not sure if this will work on .net core 3.0. But we could just disable the feature .net core if it's not possible there.

@HamletHakobyan
Copy link
Author

HamletHakobyan commented Jan 18, 2019 •
edited
Loading

Hi @batzen , good news. Don't worry about .net core. VSTO available only for .net framework.

@batzen
Copy link
Collaborator

batzen commented Jan 18, 2019

Well the code is not specifically for VSTO but enables snoop to attach to multiple app domains.

@HamletHakobyan
Copy link
Author

Hi @batzen , how are you doing? Any news on topic?

@batzen
Copy link
Collaborator

batzen commented Feb 14, 2019

I am doing fine.
Had to wait for my SettingsTransport PR to be merged as the changes i made there require changes to the cross-appdomain stuff for this issue.
Now that my PR is merged i will make the required changes for this issue till early march as i am on vacation starting tomorrow.

@HamletHakobyan
Copy link
Author

Hi @batzen , Is there any news regarding to multi-appdomain support?

@batzen
Copy link
Collaborator

batzen commented Apr 15, 2019

I am currently very busy. Will try to get this done next week.

@batzen batzen self-assigned this Apr 16, 2019
@batzen batzen added this to the 2.11 milestone Apr 16, 2019
@batzen
Copy link
Collaborator

batzen commented Jun 30, 2019

Fixed in 2.11.

@batzen batzen closed this as completed Jun 30, 2019
@miloush miloush mentioned this issue Jan 25, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@batzen batzen

Labels
enhancement
Projects
None yet
Milestone
2.11
Development

No branches or pull requests
2 participants
@batzen @HamletHakobyan