[webkit-reviews] review granted: [Bug 109952] Merge MainResourceLoader's didFinishLoading and dataReceived into DocumentLoader : [Attachment 192582] address comments

[bugzilla-daemon at webkit.org]

Antti Koivisto <koivisto at iki.fi> has granted Nate Chapin
<japhet at chromium.org>'s request for review:
Bug 109952: Merge MainResourceLoader's didFinishLoading and dataReceived into
DocumentLoader
https://bugs.webkit.org/show_bug.cgi?id=109952

Attachment 192582: address comments
https://bugs.webkit.org/attachment.cgi?id=192582&action=review

------- Additional Comments from Antti Koivisto <koivisto at iki.fi>
View in context: https://bugs.webkit.org/attachment.cgi?id=192582&action=review


> Source/WebCore/loader/DocumentLoader.cpp:330
> +    timing()->setResponseEnd(finishTime ? finishTime :
(m_timeOfLastDataReceived ? m_timeOfLastDataReceived :
monotonicallyIncreasingTime()));

A variable would be nice.