Closed
Bug 1267514
Opened 9 years ago
Closed 9 years ago
about:neterror/certerror fallback is broken (it breaks if the error code does not match an existing element)
Categories
(Firefox :: General, defect, P1)
Firefox
General
Tracking
()
Tracking | Status | |
---|---|---|
firefox47 | --- | unaffected |
firefox48 | --- | verified |
firefox49 | --- | verified |
People
(Reporter: magicp.jp, Assigned: nhnt11)
References
()
Details
(Keywords: regression, Whiteboard: [fxprivacy])
Attachments
(2 files)
(deleted),
image/png
|
Details | |
(deleted),
text/x-review-board-request
|
Gijs
:
review+
lizzard
:
approval-mozilla-aurora+
|
Details |
User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:49.0) Gecko/20100101 Firefox/49.0
Build ID: 20160425102203
Steps to reproduce:
1. Start latest Nightly
2. Go to about:neterror (or about:certerror)
Actual results:
Cannot display the problem loading page (Oops.) with correct layout.
Expected results:
Displays the problem loading page with correct layout.
Regression range:
https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=ab85f19b67b1d093e060bfe52a672a3a9995e252&tochange=9062049993e1fe1d4ca4d0c45155b6502206459c
URL: about:neterror
Has Regression Range: --- → yes
Has STR: --- → yes
status-firefox48:
--- → affected
status-firefox49:
--- → affected
Component: Untriaged → Theme
OS: Unspecified → All
Hardware: Unspecified → All
Comment 2•9 years ago
|
||
Does this happen with actual error pages (and if so, can you give an example?) or just if you manually navigate to about:neterror / about:certerror ?
Flags: needinfo?(magicp.jp)
Comment 3•9 years ago
|
||
This seems restricted to cases where we have no error-specific error message.
Nihanth, can you take this?
Component: Theme → General
Flags: needinfo?(magicp.jp) → needinfo?(nhnt11)
Summary: Cannot display the problem loading page with correct layout → about:neterror/certerror fallback is broken (it breaks if the error code does not match an existing element)
Assignee | ||
Comment 4•9 years ago
|
||
Assigning.
Assignee: nobody → nhnt11
Status: NEW → ASSIGNED
Flags: needinfo?(nhnt11)
Comment 6•9 years ago
|
||
Browser console ha this clue (maybe you've already discovered it):
> TypeError: document.getElementById(...) is null
> initPage()
> /:220
If I click the source link there, I'm taken to the getElementById call here:
> function initPage()
> {
[...]
> // if it's an unknown error or there's no title or description
> // defined, get the generic message
> var errTitle = document.getElementById("et_" + err).innerHTML;
Source link:
http://hg.mozilla.org/mozilla-central/annotate/ab0044bfa1df/browser/base/content/aboutNetError.xhtml#l220
That's from bug 1220481. I'm assuming this is a regression from that bug.
Blocks: 1220481
Keywords: regression
Comment 7•9 years ago
|
||
(In reply to Daniel Holbert [:dholbert] from comment #6)
> Browser console ha this clue (maybe you've already discovered it):
> > TypeError: document.getElementById(...) is null
> > initPage()
> > /:220
>
> If I click the source link there, I'm taken to the getElementById call here:
> > function initPage()
> > {
> [...]
> > // if it's an unknown error or there's no title or description
> > // defined, get the generic message
> > var errTitle = document.getElementById("et_" + err).innerHTML;
> Source link:
> http://hg.mozilla.org/mozilla-central/annotate/ab0044bfa1df/browser/base/
> content/aboutNetError.xhtml#l220
>
> That's from bug 1220481. I'm assuming this is a regression from that bug.
Yes, this was implied from comment #1, but I forgot to add the bug to the list of 'blocking' this bug - thanks for doing that!
Note that this is only an issue if you manually open about:neterror, not for any other actual error pages that I've seen - yet - but it could in principle also happen for "real" errors if we ever pass errors for which there is no definite message in the html page right now. I don't know if there are such errors.
I am testing manually navigate to about:neterror/about:certerror for third-party complete themes. also attack and trap sites. Anyway no layout collapse bugs in Firefox 47 and lower version.
Assignee | ||
Comment 9•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/50265/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/50265/
Attachment #8748412 -
Flags: review?(gijskruitbosch+bugs)
Comment 10•9 years ago
|
||
Comment on attachment 8748412 [details]
MozReview Request: Bug 1267514 - Fix about:neterror generic fallback text when there is no available title/description for the error. r=Gijs
https://reviewboard.mozilla.org/r/50265/#review47169
Attachment #8748412 -
Flags: review?(gijskruitbosch+bugs) → review+
Assignee | ||
Comment 11•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/9428c76cb9df77442a54cd68cae5a8426ba334a7
Bug 1267514 - Fix about:neterror generic fallback text when there is no available title/description for the error. r=Gijs
Comment 12•9 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Updated•9 years ago
|
Blocks: 1216897
Iteration: --- → 49.2 - May 23
Flags: qe-verify?
Priority: -- → P1
Whiteboard: [fxprivacy]
Comment 13•9 years ago
|
||
Comment on attachment 8748412 [details]
MozReview Request: Bug 1267514 - Fix about:neterror generic fallback text when there is no available title/description for the error. r=Gijs
Approval Request Comment
[Feature/regressing bug #]: bug 1220481
[User impact if declined]: in some edgecases network error pages don't provide any information / look broken
[Describe test coverage new/current, TreeHerder]: nope. :-(
[Risks and why]: low risk, minor change to fallback handling of network error pages
[String/UUID change made/needed]: no.
Attachment #8748412 -
Flags: approval-mozilla-aurora?
status-firefox47:
--- → unaffected
Comment on attachment 8748412 [details]
MozReview Request: Bug 1267514 - Fix about:neterror generic fallback text when there is no available title/description for the error. r=Gijs
Fix for recent regression, please uplift to aurora
Attachment #8748412 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
It would be good to verify this on aurora once it lands.
Flags: qe-verify? → qe-verify+
Updated•9 years ago
|
QA Contact: paul.silaghi
Comment 16•9 years ago
|
||
bugherder uplift |
Comment 17•8 years ago
|
||
Reproduced on Nightly 49.0a1 (2016-05-09).
Verified fixed 48.0a2 (2016-05-24), 49.0a1 (2016-05-25) Win 7.
Comment 18•4 years ago
|
||
It seems this issue is back since Firefox 84.
Also Firefox beta and Nightly are affected.
Comment 19•4 years ago
|
||
Reproduced on
Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:86.0) Gecko/20100101 Firefox/86.0 ID:20201227234826
Comment 20•4 years ago
|
||
(In reply to endor8 from comment #18)
It seems this issue is back since Firefox 84.
Also Firefox beta and Nightly are affected.
Filed bug 1684415 for this.
You need to log in
before you can comment on or make changes to this bug.
Description
•