Skip to content

Commit

Permalink
Bug 1733845: Fix Gecko Logging doc links r=nika,sylvestre DONTBUILD
Browse files Browse the repository at this point in the history
Gecko logging documentation moved earlier this year. Some docs and comments
still refer to the old URL, which now 404s.

Differential Revision: https://phabricator.services.mozilla.com/D127369
  • Loading branch information
jdpopkin committed Oct 5, 2021
1 parent 7220071 commit 10799ad
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion docs/testing-rust-code/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ To restrict logging to child processes, use `RUST_LOG_CHILD` instead of
Rust logging can also be forwarded to the [Gecko logger] for capture via
`MOZ_LOG` and `MOZ_LOG_FILE`.

[Gecko logger]: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Gecko_Logging
[Gecko logger]: /xpcom/logging.html

- When parsing modules from `MOZ_LOG`, modules containing `::` are considered
to be Rust modules. To log everything in a top-level module like
Expand Down
2 changes: 1 addition & 1 deletion dom/plugins/base/nsPluginLogging.h
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
// My favorite Win32 Example: SET MOZ_LOG_FILE=c:\temp\pluginLog.txt

// 4. For complete information see the Gecko Developer guide:
// https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Gecko_Logging
// https://firefox-source-docs.mozilla.org/xpcom/logging.html

class nsPluginLogging {
public:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ def setUp(self, *args, **kwargs):
{
"telemetry.fog.test.localhost_port": self.fog_ping_server.port,
# Enable FOG logging. 5 means "Verbose". See
# https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Gecko_Logging
# https://firefox-source-docs.mozilla.org/xpcom/logging.html
# for details.
"logging.config.clear_on_startup": False,
"logging.config.sync": True,
Expand Down

0 comments on commit 10799ad

Please sign in to comment.