native-messaging: fix Windows checker script not to fail on missing HKCU when HKLM is set #558
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
from #557 :
Motivation
i was bored
Additional details
update : ignore the following , i created a PR for that mdn/content#33211
btw , in the docs https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/Native_messaging#registry there is an incorrect info about registry keys . it says "two registry entries should be created for the messaging to work" as if "both" in
HKEY_CURRENT_USER
andHKEY_LOCAL_MACHINE
. but this is not true . only one should be . here is the proof in the Firefox source code : https://searchfox.org/mozilla-central/source/toolkit/components/extensions/NativeManifests.sys.mjs#59-83Related issues and pull requests
fixes #557