[webkit-reviews] review canceled: [Bug 110835] [Chromium] Implement target-densityDpi viewport property emulation : [Attachment 195362] A new approach suggested by aelias@

[bugzilla-daemon at webkit.org]

Mikhail Naganov <mnaganov at chromium.org> has canceled Mikhail Naganov
<mnaganov at chromium.org>'s request for review:
Bug 110835: [Chromium] Implement target-densityDpi viewport property emulation
https://bugs.webkit.org/show_bug.cgi?id=110835

Attachment 195362: A new approach suggested by aelias@
https://bugs.webkit.org/attachment.cgi?id=195362&action=review

------- Additional Comments from Mikhail Naganov <mnaganov at chromium.org>
Alexandre, thanks for your suggestions!

This is a refined version. I have moved the parsing code into
ViewportArguments. But parsing can't depend on a setting, because if the
setting will be enabled after the document has been parsed, that's too late,
and I don't want to trigger reparsing when the setting changes. So I have stuck
with a Chromium-only setting that controls, whether we are actually using the
provided value.