Closed
Bug 779838
Opened 12 years ago
Closed 12 years ago
Attr.firstChild returning NULL
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: merkinman, Unassigned)
References
Details
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:14.0) Gecko/20100101 Firefox/14.0.1
Build ID: 20120713134347
Steps to reproduce:
Visited http://www.power1051fm.com/pages/onair/breakfast-club/ that worked in previous versions of Firefox as well as other browsers
Actual results:
Images did not load and the Error Console displayed "Error: TypeError: items[i].getElementsByTagNameNS("*", "thumbnail")[0].attributes.getNamedItem("url").firstChild is null
Source File: http://www.power1051fm.com/pages/onair/breakfast-club/
Line: 470"
Expected results:
The thumbnails display like they do in other browsers as well as previous versions of Firefox
Reporter | ||
Updated•12 years ago
|
OS: Windows 7 → All
Hardware: x86_64 → All
Comment 1•12 years ago
|
||
Regression window
Good:
http://hg.mozilla.org/mozilla-central/rev/9894cd999781
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:14.0) Gecko/20120403 Firefox/14.0a1 ID:20120403002322
Bad:
http://hg.mozilla.org/mozilla-central/rev/c410b2d6d570
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:14.0) Gecko/20120403 Firefox/14.0a1 ID:20120403002822
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=9894cd999781&tochange=c410b2d6d570
Triggered by:
af7362ead6e5 Ms2ger Bug 737122 - Attr nodes shouldn't have children; r=sicking
Blocks: 737122
Status: UNCONFIRMED → NEW
Component: Untriaged → DOM: Core & HTML
Ever confirmed: true
Product: Firefox → Core
Comment 2•12 years ago
|
||
Yep, this is intentional.
Instead of
var rss_thumb = items[i].getElementsByTagNameNS('*','thumbnail')[0].attributes.getNamedItem('url').firstChild.nodeValue;
the site should use
var rss_thumb = items[i].getElementsByTagNameNS('*','thumbnail')[0].getAttribute('url');
Summary: getElementsByTagNameNS returning NULL → Attr.firstChild returning NULL
Reporter | ||
Comment 3•12 years ago
|
||
Webpage is updated with new code now. Working in FIrefox 13 and 14, as well as other browsers tested.
Comment 4•12 years ago
|
||
(In reply to :Ms2ger (Back and backlogged) from comment #2)
> Yep, this is intentional.
Marked as WONTFIX
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•