Handling of different types of empty alt

content/browser/accessibility/browser_accessibility_win.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/accessibility/browser_accessibility_win.h"
 
 #include <UIAutomationClient.h>
 #include <UIAutomationCoreApi.h>
 
 #include <algorithm>
@@ skipping 658 matching lines @@
     return E_INVALIDARG;
 
   base::string16 name_str = target->name();
   if (name_str.empty()) {
     if (target->ia2_role() == ROLE_SYSTEM_DOCUMENT && PlatformGetParent()) {
       // Hack: Some versions of JAWS crash if they get an empty name on
       // a document that's the child of an iframe, so always return a
       // nonempty string for this role.  https://crbug.com/583057
       name_str = L" ";
     } else {
-      return S_FALSE;
+      // For no accessible name, we return S_FALSE
+      // Explicitly empty names like <img alt=""> will return return ""/S_OK

[dmazzoni, 2017/05/01 17:49:23]

delete /S_OK

[aleventhal, 2017/05/17 17:45:47]

Done.

+      if (!target->HasExplicitlyEmptyName())
+        return S_FALSE;
     }
   }
 
   *name = SysAllocString(name_str.c_str());
 
   DCHECK(*name);
   return S_OK;
 }
 
 STDMETHODIMP BrowserAccessibilityWin::get_accParent(IDispatch** disp_parent) {
@@ skipping 4982 matching lines @@
   return static_cast<BrowserAccessibilityWin*>(obj);
 }
 
 const BrowserAccessibilityWin*
 ToBrowserAccessibilityWin(const BrowserAccessibility* obj) {
   DCHECK(!obj || obj->IsNative());
   return static_cast<const BrowserAccessibilityWin*>(obj);
 }
 
 }  // namespace content