Hi , Alexey,
Thanks for your comment! I am really sorry, what I used source code was
too older, which still keep the DOMWindow in the Frame, not in the
Document. That question is OK now.
Best Regards!
Hao
On 2014年09月04日 15:49, Alexey Proskuryakov wrote:
Hi!
Could you please file a bug at bugs.webkit.org? If you have a reproducible test
case where any bad behavior happens, that would be most useful.
I think that the proposed fix would break a case where we currently match
Firefox:
main.html:
---------------------
<button onclick="f()">Click</button>
<script>
function f()
{
var child = window.open("child.html");
child.navigator.foo = "bar";
child.onload = function() {
setTimeout(function() {
var w = child.frames[0];
w.navigator.foo = "bar"
child.location = "about:blank";
setTimeout(function() {
alert("w.navigator.foo: " + w.navigator.foo +
"\nw.parent.navigator.foo: " + w.parent.navigator.foo +
"\nchild.navigator.foo: " + child.navigator.foo);
}, 100);
}, 100);
}
}
</script>
---------------------
child.html:
---------------------
<iframe src="child2.html"></iframe>
---------------------
child2.html:
---------------------
<p>Hello, world!</p>
---------------------
In this test, we successfully access window.navigator of both main frame and
subframe, even though they are in page cache, and DOMWindow::navigator() has an
isCurrentlyDisplayedInFrame() check.
There was some unfinished work to make this code more reasonable, but it stopped long ago:
<https://bugs.webkit.org/show_bug.cgi?id=62054>,
<https://bugs.webkit.org/show_bug.cgi?id=68849>.
The name "isCurrentlyDisplayedInFrame" is clearly inaccurate, however it's not
clear to me whether we should be trying to make this function better match what it claims
to do, given the above. It seems that we should instead rename it, and inspect call sites
like DOMWindow::postMessageTimerFired for whether they are doing the right thing.
- Alexey
03 сент. 2014 г., в 5:29, chenhao <[email protected]> написал(а):
Hi,
We met one issue related with PostMessageTimer, it may launched while the Page
had been moved in Page Cache. After checking the implementation, we doubt this
situation should be forbid as below:
void DOMWindow::postMessageTimerFired(PostMessageTimer& timer)
{
if (!document() || !isCurrentlyDisplayedInFrame())
return;
But, unfortunately, isCurrentlyDisplayedInFrame() could not work well with
sub-frame, because of the sub-frame and its document would be kept same as
before moving in Page Cache, that means the judgement return true always for
sub-frame.
So, what I want to do is to judge inPageCache() additionally. Just like below:
bool DOMWindow::isCurrentlyDisplayedInFrame() const
{
return m_frame && m_frame->domWindow() == this &&
!m_frame->document()->inPageCache();
}
That's appreciate to get your comments!
Thanks & Best Regards!
Hao
_______________________________________________
webkit-dev mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-dev
_______________________________________________
webkit-dev mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-dev
