Bug 11674
Summary: | REGRESSION: tvguide.com XHR doesn't complete | ||
---|---|---|---|
Product: | WebKit | Reporter: | dnigdong <kyle-webkit> |
Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED FIXED | ||
Severity: | Normal | CC: | ap, dev+webkit, joost, mhaque, mjs |
Priority: | P1 | Keywords: | InRadar, NeedsReduction, Regression |
Version: | 420+ | ||
Hardware: | Mac (PowerPC) | ||
OS: | OS X 10.4 | ||
URL: | http://online.tvguide.com/listings/default.aspx |
dnigdong
You may have to log-in and specify a zip-code and such, but after that, just loading that page should show the problem.
I see the following error in the Console.app log:
[18789] http://www.tvguide.com/Listings/ line 559: TypeError: Undefined value
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Matt Lilek
Regressed between r16634 and r16680.
mhaque
After formatting through their mangled source, error occurs because selectedIndex for the select option they are looking at is returning -1. Trying to come up with a stripped down test case
function getSelectListValue(strListName, boolDefault)
{
var list = document.getElementById(strListName);
if(list != null)
{
return list[list.selectedIndex].value;
}
return boolDefault;
}
optionTimebarRepeat=getSelectListValue("selTimebarRepeat", optionTimebarRepeat);
.....
<select class=optionSelector id=selTimebarRepeat disabled onchange="javascript:setOptionsChanged()" name=selTimebarRepeat>
<option value=5>5 rows</option>
<option value=10 selected>10 rows</option>
<option value=15>15 rows</option>
<option value=20>20 rows</option>
<option value=25>25 rows</option>
<option value=30>30 rows</option>
<option value=35>35 rows</option>
<option value=40>40 rows</option>
<option value=45>45 rows</option>
<option value=50>50 rows</option>
</select>
Joost de Valk (AlthA)
mhague, is that testcase complete? if so, please attach and change the keyword of this bug to hasreduction :)
Stephanie Lewis
In Radar 4896972
Alexey Proskuryakov
The fix for <rdar://problem/4896972> landed in revision 19383 didn't fix the problem for me - the listings still don't load.
Maciej Stachowiak
For me, with 19463, the listings display but after a delay the page reloads as blank. Probably a separate bug.
Alexey Proskuryakov
I cannot reproduce the failure to load anymore.
Anders Carlsson
I can't reproduce this either, closing.