sync: Make BaseNode lookup-related Init functions return specific failures.

chrome/browser/sync/test/integration/enable_disable_test.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 "chrome/browser/sync/test/integration/sync_test.h"
 
 #include "chrome/browser/sync/internal_api/read_node.h"
 #include "chrome/browser/sync/internal_api/read_transaction.h"
 #include "chrome/browser/sync/profile_sync_service_harness.h"
 #include "sync/syncable/model_type.h"
@@ skipping 16 matching lines @@
   EnableDisableSingleClientTest() : EnableDisableTest(SINGLE_CLIENT) {}
   virtual ~EnableDisableSingleClientTest() {}
  private:
   DISALLOW_COPY_AND_ASSIGN(EnableDisableSingleClientTest);
 };
 
 bool DoesTopLevelNodeExist(sync_api::UserShare* user_share,
                            syncable::ModelType type) {
     sync_api::ReadTransaction trans(FROM_HERE, user_share);
     sync_api::ReadNode node(&trans);
-    return node.InitByTagLookup(syncable::ModelTypeToRootTag(type));
+    return node.InitByTagLookup(syncable::ModelTypeToRootTag(type)) ==
+        sync_api::BaseNode::INIT_OK;
 }
 
 IN_PROC_BROWSER_TEST_F(EnableDisableSingleClientTest, EnableOneAtATime) {
   ASSERT_TRUE(SetupClients());
 
   // Setup sync with no enabled types.
   ASSERT_TRUE(GetClient(0)->SetupSync(syncable::ModelTypeSet()));
 
   // TODO(rlarocque, 97780): It should be possible to disable notifications
   // before calling SetupSync().  We should move this line back to the top
@@ skipping 66 matching lines @@
     if (it.Get() == syncable::AUTOFILL) {
       ASSERT_FALSE(DoesTopLevelNodeExist(user_share,
                                          syncable::AUTOFILL_PROFILE));
     }
   }
 
   EnableNotifications();
 }
 
 }  // namespace