Remove IBrowserHostServices/INativeProgressPage and cleanup AppSecurityManager #10220
+68
−413
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Another batch of XBAP-related code removal; continues #9865 and previous PRs.
IBrowserHostServices
fully as it now had 1 unused enum left and that was it.INativeProgressPage
and it was used inXappLauncherApp
which is no longer present.IsAllowedScriptScheme
function inWebBrowserEvent
.AppSecurityManager
and removes the obsolete method overloads.SiteOfOriginContainer
classBooleanSwitch
behind property, make itreadonly
AppDomain.CurrentDomain.BaseDirectory
change:CurrentDomain
is a singleton and theBaseDirectory
property just redirects toAppContext.BaseDirectory
. As part of ongoing effort, I've replaced it here.Customer Impact
Decreased size of assemblies, tiny bit of perf, cleaner codebase for developers.
Regression
No.
Testing
Local build, app run.
Risk
Low, the code is a dead code or it doesn't do anything.
Microsoft Reviewers: Open in CodeFlow