Do not check against child_count() > 0, instead check if node is not empty().

Use TreeNode::empty() accessor function for checking if node has children or if
it isn't empty, instead of using child_count().

BUG=None
TEST=None

R=gbillock@chromium.org

Review URL: http://codereview.chromium.org/7775001

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

chrome/browser/ui/intents/intents_model.cc

@@ -95,7 +95,7 @@ void IntentsModel::GetIntentsTreeNodeDictionary(const IntentsTreeNode& node,
 
   if (node.Type() == IntentsTreeNode::TYPE_ORIGIN) {
     dict->SetString("site", node.GetTitle());
-    dict->SetBoolean("hasChildren", node.child_count() > 0);
+    dict->SetBoolean("hasChildren", !node.empty());
     return;
   }
 

chrome/browser/ui/webui/options/intents_settings_handler.cc

@@ -103,7 +103,7 @@ void IntentsSettingsHandler::RemoveIntent(const base::ListValue* args) {
 
 void IntentsSettingsHandler::RemoveOrigin(IntentsTreeNode* node) {
   // TODO(gbillock): This is a known batch update. Worth optimizing?
-  while (node->child_count() > 0) {
+  while (!node->empty()) {
     IntentsTreeNode* cnode = node->GetChild(0);
     CHECK(cnode->Type() == IntentsTreeNode::TYPE_SERVICE);
     ServiceTreeNode* snode = static_cast<ServiceTreeNode*>(cnode);