Remove redundant DocumentLoadTiming references.

Remove redundant DocumentLoadTiming references.

Having ScriptedIdleTaskController and its auxillary object keep a reference
to a document's DocumentLoadTiming part object is unnecessary as the deadlines
do not have to be translated wrt a Document's baseline reference time.

R=sami, rmcilroy
BUG=

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

[sof, 2015-08-27 15:00:47]

sigbjornf@opera.com changed reviewers:
+ alexclarke@chromium.org, haraken@chromium.org, skyostil@chromium.org

[sof, 2015-08-27 15:00:47]

please take a look.

[Sami, 2015-08-27 15:18:21]

I'm not sure how the ms => s unit change is related. Should that be in a
different patch?

[sof, 2015-08-27 15:27:22]

On 2015/08/27 15:18:21, Sami wrote:
> I'm not sure how the ms => s unit change is related. Should that be in a
> different patch?

The deadlineMillis passed in to IdleCallbackDeadline wasn't a deadline, but time
remaining (in MS) -- if I understand the terminology you're using. So I made it
be a deadline here, and avoided the premultiplication up to MS before passing it
in, as that didn't make too much sense to do any longer.

[sof, 2015-08-27 15:28:26]

sigbjornf@opera.com changed reviewers:
+ rmcilroy@chromium.org

[sof, 2015-08-27 15:28:26]

+rmcilroy@

[Sami, 2015-08-27 15:42:46]

Ah, right, I agree that keeping the deadline time (as opposed to the time
remaining) in seconds makes more sense.

Non-owner lgtm.

[rmcilroy, 2015-08-27 16:08:46]

Thanks for the fix (originally the m_timing object was required because we were
returning a DOMHighPrecisionTime and so needed to convert the deadline, but in a
last minute change we moved to returning a timeRemaining milliseconds instead,
so you're right that it is no longer required anymore).

lgtm, thanks!

[sof, 2015-08-27 18:34:21]

On 2015/08/27 16:08:46, rmcilroy wrote:
> Thanks for the fix (originally the m_timing object was required because we
were
> returning a DOMHighPrecisionTime and so needed to convert the deadline, but in
a
> last minute change we moved to returning a timeRemaining milliseconds instead,
> so you're right that it is no longer required anymore).
> 
> lgtm, thanks!

Thanks for the explanation & confirmation. Knowing how clearly you guys reason
around timing details, I really wasn't sure if I hadn't overlooked something.

[sof, 2015-08-27 18:46:20]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2015-08-27 18:46:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1316323002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1316323002/40001

[commit-bot: I haz the power, 2015-08-27 20:47:49]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=201338

[haraken, 2015-08-27 23:38:39]

LGTM