Native Android accessibility.

content/browser/renderer_host/render_view_host_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/browser/renderer_host/render_view_host_impl.h"
 
 #include <set>
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 1891 matching lines @@
 
 void RenderViewHostImpl::OnAccessibilityNotifications(
     const std::vector<AccessibilityHostMsg_NotificationParams>& params) {
   if (view_ && !is_swapped_out_)
     view_->OnAccessibilityNotifications(params);
 
   for (unsigned i = 0; i < params.size(); i++) {
     const AccessibilityHostMsg_NotificationParams& param = params[i];
     AccessibilityNotification src_type = param.notification_type;
 
-    if ((src_type == AccessibilityNotificationLayoutComplete ||
-         src_type == AccessibilityNotificationLoadComplete) &&
-        save_accessibility_tree_for_testing_) {
-      MakeAccessibilityNodeDataTree(param.nodes, &accessibility_tree_);
+    bool is_about_blank = false;
+    for (unsigned j = 0; j < param.nodes.size(); j++) {
+      std::map<AccessibilityNodeData::StringAttribute, string16>::const_iterator iter;

[David Trainor- moved to gerrit, 2013/06/04 20:27:18]

line too long.

[dmazzoni, 2013/06/07 20:23:16]

Done.

+      iter = param.nodes[j].string_attributes.find(
+          AccessibilityNodeData::ATTR_DOC_URL);
+      if (iter != param.nodes[j].string_attributes.end()) {
+        string16 doc_url = iter->second;
+        if (doc_url == ASCIIToUTF16("about:blank"))

[David Trainor- moved to gerrit, 2013/06/04 20:27:18]

I feel like about:blank a constant somewhere.  I don't remember the place
though.

[dmazzoni, 2013/06/07 20:23:16]

Done.

+          is_about_blank = true;

[David Trainor- moved to gerrit, 2013/06/04 20:27:18]

break here?  Could also just put the logic in here and break.  No need to save
the bool?

[dmazzoni, 2013/06/07 20:23:16]

We'd need to break out of one for loop and continue from the other, which I'm
not sure would be more readable.

+      }
     }
 
-    if (src_type == AccessibilityNotificationLayoutComplete) {
-      accessibility_layout_callback_.Run();
-    } else if (src_type == AccessibilityNotificationLoadComplete) {
-      accessibility_load_callback_.Run();
-    } else {
-      accessibility_other_callback_.Run();
+    if (!is_about_blank) {
+      if ((src_type == AccessibilityNotificationLayoutComplete ||
+           src_type == AccessibilityNotificationLoadComplete) &&
+          save_accessibility_tree_for_testing_) {
+        MakeAccessibilityNodeDataTree(param.nodes, &accessibility_tree_);
+      }
+
+      if (src_type == AccessibilityNotificationLayoutComplete) {
+        accessibility_layout_callback_.Run();
+      } else if (src_type == AccessibilityNotificationLoadComplete) {
+        accessibility_load_callback_.Run();
+      } else {
+        accessibility_other_callback_.Run();
+      }
     }
   }
 
   Send(new AccessibilityMsg_Notifications_ACK(GetRoutingID()));
 }
 
 void RenderViewHostImpl::OnScriptEvalResponse(int id,
                                               const base::ListValue& result) {
   const base::Value* result_value;
   if (!result.Get(0, &result_value)) {
@@ skipping 155 matching lines @@
   const std::vector<base::FilePath>& file_paths = state.GetReferencedFiles();
   for (std::vector<base::FilePath>::const_iterator file = file_paths.begin();
        file != file_paths.end(); ++file) {
     if (!policy->CanReadFile(GetProcess()->GetID(), *file))
       return false;
   }
   return true;
 }
 
 }  // namespace content