Closed Bug 1177904 Opened 7 years ago Closed 2 years ago

[eslint: no-undef] Investigate `"aText" is not defined`

Categories

(Firefox for Android Graveyard :: General, defect)

All
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: mcomella, Unassigned)

References

Details

via eslint:

components/PromptService.js
  399:60  error  "aText" is not defined  no-undef
  401:49  error  "aText" is not defined  no-undef

via bug 1170804 comment 20:

> > I imagine aText is supposed to be in the method header, but I can't find the
> > idl definition.
> 
> Looks like text should be a parameter here...
> http://mxr.mozilla.org/mozilla-central/source/netwerk/base/nsIAuthPrompt.idl#31
> 
> wesj, is there just a typo in here that we're missing an aText parameter?
> http://mxr.mozilla.org/mozilla-central/source/mobile/android/components/PromptService.js#384

I added an ignore in bug 1177774 but we should figure out what's going on here.
Flags: needinfo?(wjohnston)
Yeah, looks like a typo. There are other auth prompt interfaces in here, so I'm guessing we just never/rarely hit this.
Flags: needinfo?(wjohnston)
Blocks: 1424996

This file has gone away now.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WORKSFORME
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.