Add loadedNonEmptyDocument flag in iframe for Resource Timing

Add loadedNonEmptyDocument flag in iframe for Resource Timing

Sometimes iframe src is blank in markup, and then will be set a real
URL via script, Resource Timing would like to populate resource fetched from
the real URL, but avoid the self navigation after that, so this change add
a flag loadedNonEmptyDocument to achieve that.

Contributed by pan.deng@intel.com

BUG=236180
TBR=japhet

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=150353

[pan deng, 2013-05-09 03:24:46]

Hi japhet,

Trybot and CQ don't work for me in CL https://codereview.chromium.org/14054023/,
I consulted dpranke, the reason may be pan.deng@intel.com has no read-only svn
access.

So I created this issue by chromium.org account(just with read-only svn access),
please review. :)


thanks!
Pan

[Nate Chapin, 2013-05-09 16:07:23]

lgtm

[commit-bot: I haz the power, 2013-05-09 16:07:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pan.deng@chromium.org/14632007/1

[commit-bot: I haz the power, 2013-05-09 18:03:36]

Retried try job too often on win_layout_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_layout...

[commit-bot: I haz the power, 2013-05-09 23:43:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pan.deng@chromium.org/14632007/1

[commit-bot: I haz the power, 2013-05-10 03:09:32]

Retried try job too often on win_layout_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_layout...

[commit-bot: I haz the power, 2013-05-11 10:28:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pan.deng@chromium.org/14632007/1

[pan deng, 2013-05-14 12:40:55]

James and Tony,

This patch failed on win layout tests because two reasons.
1) php header("timing-allow-origin: $origin") should be in the first section,
otherwise it won't be sent to client in strict mode (php-cgi for windows
lighttpd doesn't enable buffering header mechnasim).
2) PerformanceResourceTiming::resourceTimeToDocumentMilliseconds assumes delta
is positive millisecond, while I observed delta sometimes is 0 in win layout
test(too light weight? or time accurate not good enough in win? it never
happened if use chromium browser), as dns_start recorded by TimeTicks:Now() is
quite near request_time.So I suggest we assert it is not negative, and accept
the delta 0(less than 1 millisecond delta).

please reiview, thanks!

Pan

[James Simonsen, 2013-05-14 17:36:43]

lgtm

[commit-bot: I haz the power, 2013-05-14 17:37:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pan.deng@chromium.org/14632007/38001

[James Simonsen, 2013-05-14 17:38:22]

BTW,

On 2013/05/14 12:40:55, pan deng wrote:
> or time accurate not good enough in win?

It's this one. We only have 1ms resolution on Windows.

[commit-bot: I haz the power, 2013-05-14 20:43:22]

Change committed as 150353