Closed Bug 1484925 Opened 3 years ago Closed 3 years ago

UseCounter documentation links to non-existing file


(Toolkit :: Telemetry, enhancement, P3)




Tracking Status
firefox63 --- fixed


(Reporter: janerik, Assigned: arshadkazmi42, Mentored)


(Whiteboard: [good first bug][lang=rst])


(1 file)

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.

Mentor: jrediger
Whiteboard: [good first bug][lang=rst]
Priority: -- → P3
ServoCSSPropList.h is available here:

:emilio: Is that the best file to link to from the documentation?
Flags: needinfo?(emilio)
Yeah, that one should work. Thanks!
Flags: needinfo?(emilio)
Is it open for grab? I am interested to take this up.
Flags: needinfo?(jrediger)
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.


Please ping me if you need any additional help.
Assignee: nobody → arshadkazmi42
Flags: needinfo?(jrediger)
Below searchfox url needs to be updated in that?


Or is there any other production URL which needs to be used?
Flags: needinfo?(jrediger)
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
Pushed code to Phabricator. And added you as reviewer
Flags: needinfo?(jrediger)
Comment on attachment 9004317 [details]
Bug 1484925 - UseCounter Broken Link Fixed.

Jan-Erik Rediger [:janerik] has approved the revision.
Attachment #9004317 - Flags: review+
Pushed by
UseCounter Broken Link Fixed. r=janerik
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.