1525189 - Link definitions in WebIDL files to uses in JS

Status: REOPENED

(Webtools :: Searchfox, defect)

Description

[Botond Ballo [:botond]]

STR:

1. Load https://searchfox.org/mozilla-central/source/mobile/android/components/SessionStore.js
2. Click on restoreScrollPosition on this line.
3. Click "Go to definition of restoreScrollPosition"

Expected results:

Searchfox navigates to the definition of the function in question in SessionStoreUtils.webidl.

Actual results:

Searchfox navigates to an unrelated definition of the same name in aboutPolicies.js.

Comment 1

[Andrew McCreight [:mccr8]]

That WebIDL file doesn't seem to have any definition information hooked up for it. If you click on anything, it only has the "search for substring" option. I don't recall if that's expected behavior or maybe something broke.

Comment 2

[Andrew McCreight [:mccr8]]

I think this is a dupe of bug 1416899, or at least depends on it.

Comment 3

[Andrew McCreight [:mccr8]]

I suppose this is about the JS part of WebIDL, and that bug is talking about the C++ part.

Comment 4

[Andrew McCreight [:mccr8]]

Offhand, this seems like it would be a lot trickier than the C++ part of it, because the analysis would have to understand what implementations are defined on the global. It would be quite cool, though, if you could click on some random DOM API and get to the C++ implementation in a few clicks (with the combination of both of these bugs).