[webkit-reviews] review granted: [Bug 74188] Web Inspector: Generated HAR is missing pages.startedDateTime : [Attachment 119236] Patch

[bugzilla-daemon at webkit.org]

Pavel Feldman <pfeldman at chromium.org> has granted Andrey Kosyakov
<caseq at chromium.org>'s request for review:
Bug 74188: Web Inspector: Generated HAR is missing pages.startedDateTime
https://bugs.webkit.org/show_bug.cgi?id=74188

Attachment 119236: Patch
https://bugs.webkit.org/attachment.cgi?id=119236&action=review

------- Additional Comments from Pavel Feldman <pfeldman at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=119236&action=review


> LayoutTests/http/tests/inspector/inspector-test.js:112
> +InspectorTest.formatAsTypeName = function(value)

I don't think firnatters should be defined on the InspectorTest object - use a
nested object / namespace instead.

> LayoutTests/http/tests/inspector/inspector-test.js:135
> +	   if (customFormatters && typeof customFormatters[prop] === "string")
{

Why do we have this distinction now that there is a default formatting?