Replaced 0 with nullptr in WebCore/accessibility.
[WebKit-https.git] / Source / WebCore / accessibility / atk / WebKitAccessibleInterfaceDocument.cpp
index dac1bbbaa11aee5363529fa54d65b1499d28e3cb..6752d12d323b290cc15c113a1d4e1348b06f0dc8 100644 (file)
@@ -78,12 +78,18 @@ static const gchar* documentAttributeValue(AtkDocument* document, const gchar* a
 
 static const gchar* webkitAccessibleDocumentGetAttributeValue(AtkDocument* document, const gchar* attribute)
 {
+    g_return_val_if_fail(ATK_IS_DOCUMENT(document), 0);
+    returnValIfWebKitAccessibleIsInvalid(WEBKIT_ACCESSIBLE(document), 0);
+
     return documentAttributeValue(document, attribute);
 }
 
 static AtkAttributeSet* webkitAccessibleDocumentGetAttributes(AtkDocument* document)
 {
-    AtkAttributeSet* attributeSet = 0;
+    g_return_val_if_fail(ATK_IS_DOCUMENT(document), 0);
+    returnValIfWebKitAccessibleIsInvalid(WEBKIT_ACCESSIBLE(document), 0);
+
+    AtkAttributeSet* attributeSet = nullptr;
     const gchar* attributes[] = { "DocType", "Encoding", "URI" };
 
     for (unsigned i = 0; i < G_N_ELEMENTS(attributes); i++) {
@@ -97,12 +103,15 @@ static AtkAttributeSet* webkitAccessibleDocumentGetAttributes(AtkDocument* docum
 
 static const gchar* webkitAccessibleDocumentGetLocale(AtkDocument* document)
 {
-    // TODO: Should we fall back on lang xml:lang when the following comes up empty?
-    String language = core(document)->language();
-    if (!language.isEmpty())
-        return cacheAndReturnAtkProperty(ATK_OBJECT(document), AtkCachedDocumentLocale, language);
-
-    return 0;
+    g_return_val_if_fail(ATK_IS_DOCUMENT(document), 0);
+    returnValIfWebKitAccessibleIsInvalid(WEBKIT_ACCESSIBLE(document), 0);
+
+    // The logic to resolve locale has been moved to
+    // AtkObject::get_object_locale() virtual method. However, to avoid breaking
+    // clients expecting the deprecated AtkDocumentIface::get_document_locale()
+    // to be overriden, method is kept and chained up to
+    // AtkObject::get_object_locale(). <https://bugs.webkit.org/show_bug.cgi?id=115647>
+    return atk_object_get_object_locale(ATK_OBJECT(document));
 }
 
 void webkitAccessibleDocumentInterfaceInit(AtkDocumentIface* iface)