Pass accessible location change messages to the automation API.

content/renderer/accessibility/render_accessibility_impl.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/renderer/accessibility/render_accessibility_impl.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <queue>
@@ skipping 377 matching lines @@
     WebFloatRect bounds_in_container;
     SkMatrix44 container_transform;
     obj.getRelativeBounds(
         offset_container, bounds_in_container, container_transform);
     ui::AXRelativeBounds new_location;
     new_location.offset_container_id = offset_container.axID();
     new_location.bounds = bounds_in_container;
     if (!container_transform.isIdentity())
       new_location.transform = base::WrapUnique(
           new gfx::Transform(container_transform));
-    if (iter != locations_.end() && iter->second != new_location) {

[aboxhall, 2016/08/22 19:48:38]

Why is this no longer necessary?

[dmazzoni, 2016/08/22 21:29:14]

A few lines above, we have:

if (iter == locations_.end())
  continue;

So it doesn't seem to be necessary to check it again.

+    if (iter->second != new_location) {
       AccessibilityHostMsg_LocationChangeParams message;
       message.id = id;
       message.new_location = new_location;
       messages.push_back(message);
     }
 
     // Save the new location.
     new_locations[id] = new_location;
 
     // Explore children of this object.
@@ skipping 280 matching lines @@
       size_t new_count = pdf_update.nodes.size();
       update->nodes.resize(old_count + new_count);
       for (size_t i = 0; i < new_count; ++i)
         update->nodes[old_count + i] = pdf_update.nodes[i];
       break;
     }
   }
 }
 
 }  // namespace content