[webkit-reviews] review denied: [Bug 15062] Implement document.compatMode : [Attachment 16147] implementation in the Document interface

[bugzilla-daemon at webkit.org]

Sam Weinig <sam at webkit.org> has denied Vincent Ricard <magic at magicninja.org>'s
request for review:
Bug 15062: Implement document.compatMode
http://bugs.webkit.org/show_bug.cgi?id=15062

Attachment 16147: implementation in the Document interface
http://bugs.webkit.org/attachment.cgi?id=16147&action=edit

------- Additional Comments from Sam Weinig <sam at webkit.org>
The "{" should be on a new line.
+String Document::compatMode() const {
+    return inCompatMode() ? "BackCompat" : "CSS1Compat";
+}

I don't think there is any reason for [ConvertNullStringTo=Undefined] as null
is not a possible return value.
+	 readonly attribute [ConvertNullStringTo=Undefined] DOMString
compatMode;

Also, does this attribute make sense for Documents other HTML documents?  If
not, can we put this in the HTMLDocument?  r- for now.