Bug 13444 - REGRESSION: "No parent frame" error messages during layout tests
Summary: REGRESSION: "No parent frame" error messages during layout tests
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: 523.x (Safari 3)
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar, Regression
: 13330 (view as bug list)
Depends on:
Blocks:
 
Reported: 2007-04-22 01:42 PDT by Alexey Proskuryakov
Modified: 2007-07-15 23:34 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Proskuryakov 2007-04-22 01:42:15 PDT
I haven't investigated these.

fast/dom/Window ..........ERROR: No parent frame in transitionToCommitted:, FrameLoadTypeInternal
(/Users/ap/WebKit/WebCore/loader/FrameLoader.cpp:4054 updateHistoryForInternalLoad)
.....ERROR: No parent frame in transitionToCommitted:, FrameLoadTypeInternal
(/Users/ap/WebKit/WebCore/loader/FrameLoader.cpp:4054 updateHistoryForInternalLoad)
.ERROR: No parent frame in transitionToCommitted:, FrameLoadTypeInternal
(/Users/ap/WebKit/WebCore/loader/FrameLoader.cpp:4054 updateHistoryForInternalLoad)
..ERROR: No parent frame in transitionToCommitted:, FrameLoadTypeInternal
(/Users/ap/WebKit/WebCore/loader/FrameLoader.cpp:4054 updateHistoryForInternalLoad)
.....
Comment 1 Darin Adler 2007-04-23 08:40:24 PDT
<rdar://problem/5153031>
Comment 2 Alexey Proskuryakov 2007-04-29 11:05:31 PDT
*** Bug 13330 has been marked as a duplicate of this bug. ***
Comment 3 Maxime BRITTO 2007-06-11 07:25:14 PDT
The layout test affected by theses logs is setting-properties-on-closed-window.html which :
    1 -  creates a variable (w) and give it a new page : w = open('about:blank');
    2 -  closes this page : w.close();
    3 -  try to access the variable to test if it crashes : w.status = i;

At step 1 : the webcore handle the loading as a FrameLoadTypeStandard an there is no problem.
At step 3 : the webcore loads the formely closed page as a FrameLoadTypeInternal an when it's trying to update history for internal loads the error is logged.

I don't know why it is forbidden to load an empty page internaly. There is a comment in the code (just before the log expression) which is mentionning the bug 3556159 (rdar?) as a justification.

I tried to replace the about:blank by http://www.google.fr in the setting-properties-on-closed-window.html and it works fine.
Comment 4 Maxime BRITTO 2007-06-12 06:50:41 PDT
(In reply to comment #3)
> I don't know why it is forbidden to load an empty page internaly. There is a
> comment in the code (just before the log expression) which is mentionning the
> bug 3556159 (rdar?) as a justification.
> 
> I tried to replace the about:blank by http://www.google.fr in the
> setting-properties-on-closed-window.html and it works fine.
> 
I came back on this and I discovered that the frame loaded Internaly with no parent is not necessarily empty. 
The comment in the code was relative to frame based web pages as sub-frame loading was the only user of FrameLoadTypeInternal. But JS uses it when it reloads a page to access a variable (like in the test)

There is still one problem : this rdar bug number in the code comments.
Comment 5 mitz 2007-07-15 23:34:33 PDT
Fixed in <http://trac.webkit.org/projects/webkit/changeset/24299>.