Assert when loading websites in windows build (debug mode)

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 5432 matching lines @@
       if (aria_role == L"group" || html_tag == L"fieldset") {
         ia_role = ROLE_SYSTEM_GROUPING;
       } else if (html_tag == L"li") {
         ia_role = ROLE_SYSTEM_LISTITEM;
         ia_state |= STATE_SYSTEM_READONLY;
       } else {
         if (html_tag.empty())
           role_name = L"div";
         else
           role_name = html_tag;
+        ia_role = ROLE_SYSTEM_GROUPING;

[dmazzoni, 2017/03/10 21:35:23]

This one shouldn't be needed because the role_name
will always get something nonempty

[josyula, 2017/03/11 05:38:17]

Ok.

         ia2_role = IA2_ROLE_SECTION;
       }
       break;
     }
     case ui::AX_ROLE_HEADING:
       role_name = html_tag;
+      ia_role = ROLE_SYSTEM_GROUPING;

[dmazzoni, 2017/03/10 21:35:23]

How about we do this only if html_tag is empty.

Normally it won't be empty.

[josyula, 2017/03/11 05:38:17]

Ok sure. I will incorporate it.

       ia2_role = IA2_ROLE_HEADING;
       break;
     case ui::AX_ROLE_IFRAME:
       ia_role = ROLE_SYSTEM_DOCUMENT;
       ia2_role = IA2_ROLE_INTERNAL_FRAME;
       ia_state = STATE_SYSTEM_READONLY;
       break;
     case ui::AX_ROLE_IFRAME_PRESENTATIONAL:
       ia_role = ROLE_SYSTEM_GROUPING;
       break;
@@ skipping 317 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