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

Still need to investigate when this might happen and how to fix it,
but this will allow it to just break accessibility rather than crashing
the whole browser in the meantime.

BUG=161726


Review URL: https://chromiumcodereview.appspot.com/11787024

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@175470 0039d316-1c4b-4281-b951-d872f2087c98

content/browser/accessibility/browser_accessibility_manager.cc

@@ -356,7 +356,8 @@ BrowserAccessibility* BrowserAccessibilityManager::CreateAccessibilityTree(
   // 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;
   }