Summary: | REGRESSION (r20972): Wall Street Journal pages replaced by advertisements | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Geoffrey F. Green <geoff-public> | ||||||||
Component: | DOM | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | ap, darin, gibbonsb | ||||||||
Priority: | P1 | Keywords: | InRadar, NeedsReduction, Regression | ||||||||
Version: | 523.x (Safari 3) | ||||||||||
Hardware: | Mac | ||||||||||
OS: | OS X 10.4 | ||||||||||
URL: | http://wsj.com | ||||||||||
Attachments: |
|
Description
Geoffrey F. Green
2007-04-23 19:07:38 PDT
Created attachment 14151 [details]
Screenshot of WSJ home page rendered on latest nightly
Created attachment 14152 [details]
Screenshot of part of WSJ home page rendered on 419.3
The screenshot shows the advertising block which is near the bottom of the WSJ home page.
I suspect <http://trac.webkit.org/projects/webkit/changeset/20972> (<rdar://problem/4671964> document property on <iframe> elements causes problems for SAP application). The problem seems to be that contentDocument and contentWindow are both undefined on new frame elements until they've done enough loading to go from the provisional state to the committed state. There's code that looks at contentDocument and contentWindow first, then falls back to other properties, including document. It would work fine if contentDocument or contentWindow was defined. The function is changeIframeSrc and it's in http://online.wsj.com/javascript/commonFunctions.js *** Bug 13590 has been marked as a duplicate of this bug. *** Fixed in r21368 (change 21367) except that the browser history includes, as a separate entry, the iframe that previously replaced the document. In other words, using the sample error, the proper WSJ page is displayed, but the history list includes both the WSJ page and the title of the iframe page. Selecting the "iframe entry" from the history list sends you to the iframe. To clarify, see image of history list below. Created attachment 14475 [details] Screenshot of history list from r21368 (In reply to comment #10) > Fixed in r21368 (change 21367) except that the browser history includes, as a > separate entry, the iframe that previously replaced the document. In other > words, using the sample error, the proper WSJ page is displayed, but the > history list includes both the WSJ page and the title of the iframe page. > Selecting the "iframe entry" from the history list sends you to the iframe. To > clarify, see image of history list below. Lets make a new bug report about this smaller anomaly so we can close this one and put the focus clearly on understanding the cause of the remaining behavior. (In reply to comment #12) > Lets make a new bug report about this smaller anomaly so we can close this one > and put the focus clearly on understanding the cause of the remaining behavior. Followup is Bug 13686. Closing this bug per Comment #10. |