views/bookmarks: Fix memory leak in BookmarkEditorView.

chrome/browser/ui/views/bookmarks/bookmark_editor_view_unittest.cc

 // Copyright (c) 2011 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 <string>
 
 #include "base/message_loop.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/bookmarks/bookmark_model.h"
@@ skipping 51 matching lines @@
                     BookmarkEditor::Configuration configuration) {
     editor_.reset(new BookmarkEditorView(profile, parent, details,
                                          configuration));
   }
 
   void SetTitleText(const std::wstring& title) {
     editor_->title_tf_.SetText(title);
   }
 
   void SetURLText(const std::wstring& text) {
-    editor_->url_tf_.SetText(text);
+    if (editor_->details_.type != BookmarkEditor::EditDetails::NEW_FOLDER)
+      editor_->url_tf_->SetText(text);
   }
 
   void ApplyEdits(BookmarkEditorView::EditorNode* node) {
     editor_->ApplyEdits(node);
   }
 
   BookmarkEditorView::EditorNode* AddNewFolder(
       BookmarkEditorView::EditorNode* parent) {
     return editor_->AddNewFolder(parent);
   }
 
   bool URLTFHasParent() {
-    return editor_->url_tf_.parent();
+    if (editor_->details_.type == BookmarkEditor::EditDetails::NEW_FOLDER)
+      return false;
+    return editor_->url_tf_->parent();
   }
 
   MessageLoopForUI message_loop_;
   BrowserThread ui_thread_;
   BrowserThread file_thread_;
 
   BookmarkModel* model_;
   scoped_ptr<TestingProfile> profile_;
 
  private:
@@ skipping 263 matching lines @@
   EXPECT_EQ(BookmarkNode::FOLDER, new_node->type());
   EXPECT_EQ(ASCIIToUTF16("new_F"), new_node->GetTitle());
   // The node should have one child.
   ASSERT_EQ(1, new_node->child_count());
   const BookmarkNode* new_child = new_node->GetChild(0);
   // Make sure the child url/title match.
   EXPECT_EQ(BookmarkNode::URL, new_child->type());
   EXPECT_EQ(WideToUTF16Hack(details.urls[0].second), new_child->GetTitle());
   EXPECT_EQ(details.urls[0].first, new_child->url());
 }