Changeset 130191 in webkit

Timestamp:

Oct 2, 2012, 11:51:35 AM (13 years ago)

Author:

commit-queue@webkit.org

Message:

WebDocument doesn't export document language to outside users.
​https://bugs.webkit.org/show_bug.cgi?id=98066

Patch by Brian White <​bcwhite@chromium.org> on 2012-10-02
Reviewed by Adam Barth.

Chromium uses the document language to offer translation. It's been doing
its own detection of this property but it would be better to get it from
existing extraction code.

• public/WebDocument.h:

(WebDocument):

• src/WebDocument.cpp:

(WebKit::WebDocument::contentLanguage):
(WebKit):

Location:

trunk/Source/WebKit/chromium

Files:

• ChangeLog (modified) (1 diff)
• public/WebDocument.h (modified) (1 diff)
• src/WebDocument.cpp (modified) (1 diff)

trunk/Source/WebKit/chromium/ChangeLog

@@ +1 @@
+2012-10-02  Brian White  <bcwhite@chromium.org>
+
+        WebDocument doesn't export document language to outside users.
+        https://bugs.webkit.org/show_bug.cgi?id=98066
+
+        Reviewed by Adam Barth.
+
+        Chromium uses the document language to offer translation.  It's been doing
+        its own detection of this property but it would be better to get it from
+        existing extraction code.
+
+        * public/WebDocument.h:
+        (WebDocument):
+        * src/WebDocument.cpp:
+        (WebKit::WebDocument::contentLanguage):
+        (WebKit):
+
 2012-10-02  Vangelis Kokkevis  <vangelis@chromium.org>
 

trunk/Source/WebKit/chromium/public/WebDocument.h

@@ -80 +80 @@
 
     WEBKIT_EXPORT WebString encoding() const;
+    WEBKIT_EXPORT WebString contentLanguage() const;
 
     // The url of the OpenSearch Desription Document (if any).

trunk/Source/WebKit/chromium/src/WebDocument.cpp

@@ -82 +82 @@
 }
 
+WebString WebDocument::contentLanguage() const
+{
+    return constUnwrap<Document>()->contentLanguage();
+}
+
 WebURL WebDocument::openSearchDescriptionURL() const
 {