1484925 - UseCounter documentation links to non-existing file

Status: RESOLVED FIXED

(Toolkit :: Telemetry, enhancement, P3)

Description

[Jan-Erik Rediger [:janerik]]

Currently the Use Counter documentation[1] links to a file called `nsCSSPropList.h`[2].
However this file does not exist anymore.

I think the generated `ServoCSSPropList.h` is its replacement (however, that file is not available in searchfox or dxr)

We should fix the link.


[1]: https://firefox-source-docs.mozilla.org/toolkit/components/telemetry/telemetry/collection/use-counters.html#css-properties
[2]: https://dxr.mozilla.org/mozilla-central/source/layout/style/nsCSSPropList.h

Comment 1

[Jan-Erik Rediger [:janerik]]

ServoCSSPropList.h is available here: https://searchfox.org/mozilla-central/source/__GENERATED__/layout/style/ServoCSSPropList.h

:emilio: Is that the best file to link to from the documentation?

Comment 2

[Emilio Cobos Álvarez (:emilio)]

Yeah, that one should work. Thanks!

Comment 3

[Arshad Kazmi [:arshadkazmi42]]

Is it open for grab? I am interested to take this up.

Comment 4

[Jan-Erik Rediger [:janerik]]

Yes, I just assigned the bug to you.

The task is rather simple:
Exchange the link in the use counter documentation[1]  with the above link to the ServoCSSPropList.h file.

Once you've done that, compile the documentation with `./mach doc`, open it in a browser, go to the Use Counter documentation and check that the link works and everything is correct.

[1]: https://searchfox.org/mozilla-central/rev/e126996d9b0a3d7653de205898517f4f5b632e7f/toolkit/components/telemetry/docs/collection/use-counters.rst#52

Please ping me if you need any additional help.

Comment 5

[Arshad Kazmi [:arshadkazmi42]]

Below searchfox url needs to be updated in that?

`https://searchfox.org/mozilla-central/source/__GENERATED__/layout/style/ServoCSSPropList.h`

Or is there any other production URL which needs to be used?

Comment 6

[Arshad Kazmi [:arshadkazmi42]]

So. I found a similar type of url for other link. So adding this file and pushing code to Phabricator for review.

Let me know if that url is different & needs to be changed

Comment 7

[Arshad Kazmi [:arshadkazmi42]]

Attachment: Bug 1484925 - UseCounter Broken Link Fixed.

Comment 8

[Arshad Kazmi [:arshadkazmi42]]

Pushed code to Phabricator. And added you as reviewer

https://phabricator.services.mozilla.com/D4363

Comment 9

[Jan-Erik Rediger [:janerik]]

Comment on attachment 9004317 [details]
Bug 1484925 - UseCounter Broken Link Fixed.

Jan-Erik Rediger [:janerik] has approved the revision.

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d214235eef08
UseCounter Broken Link Fixed. r=janerik

Comment 11

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/d214235eef08