Replace NOTREACHED that was getting triggered with LOG(WARNING).

content/browser/accessibility/browser_accessibility_manager.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_manager.h"
 
 #include "base/logging.h"
 #include "content/browser/accessibility/browser_accessibility.h"
 #include "content/browser/accessibility/browser_accessibility_state_impl.h"
 #include "content/common/accessibility_messages.h"
@@ skipping 338 matching lines @@
 
   // If the node has changed roles, don't reuse a BrowserAccessibility
   // object, that could confuse a screen reader.
   // TODO(dtseng): Investigate when this gets hit; See crbug.com/93095.
   DCHECK(!instance || instance->role() == src.role);
 
   // If we're reusing a node, it should already be detached from a parent
   // and any children. If not, that means we have a serious bug somewhere,
   // like the same child is reachable from two places in the same tree.
   if (instance && (instance->parent() != NULL || instance->child_count() > 0)) {
-    NOTREACHED();
+    // TODO(dmazzoni): investigate this: http://crbug.com/161726
+    LOG(WARNING) << "Reusing node that wasn't detached from parent";
     instance = NULL;
   }
 
   if (instance) {
     // If we're reusing a node, update its parent and increment its
     // reference count.
     instance->UpdateParent(parent, index_in_parent);
     instance->InternalAddReference();
     send_show_events = false;
   } else {
@@ skipping 25 matching lines @@
   // that subtree.
   if (send_show_events)
     NotifyAccessibilityEvent(AccessibilityNotificationObjectShow, instance);
 
   instance->PostInitialize();
 
   return instance;
 }
 
 }  // namespace content