[Mac] Don't crash when dragging and dropping bookmarks within a bookmarks folder.

chrome/browser/ui/cocoa/bookmarks/bookmark_bar_folder_controller_unittest.mm

 // 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.
 
 #import <Cocoa/Cocoa.h>
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_nsobject.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/bookmarks/bookmark_model.h"
@@ skipping 578 matching lines @@
   while ([bbfc canScrollDown]) {
     [bbfc performOneScroll:-scrollOneBookmark];
     --buttonCounter;
   }
   button = [buttons objectAtIndex:buttonCounter + 3];
   [folder deleteBookmark:button];
   newTop = [menuWindow frame].origin.y + NSHeight([menuWindow frame]);
   EXPECT_CGFLOAT_EQ(oldTop - bookmarks::kScrollWindowVerticalMargin, newTop);
 }
 
+// Make sure that we return the correct browser window.
+TEST_F(BookmarkBarFolderControllerTest, BrowserWindow) {
+  scoped_nsobject<BookmarkBarFolderController> controller;
+  controller.reset(SimpleBookmarkBarFolderController());

[Robert Sesek, 2011/10/25 14:51:05]

Merge this line with the previous?

[Ilya Sherman, 2011/10/25 19:20:54]

Done.  (Was copy/pasted.)

+  EXPECT_EQ([bar_ browserWindow], [controller browserWindow]);
+}
+
 @interface FakedDragInfo : NSObject {
 @public
   NSPoint dropLocation_;
   NSDragOperation sourceMask_;
 }
 @property (nonatomic, assign) NSPoint dropLocation;
 - (void)setDraggingSourceOperationMask:(NSDragOperation)mask;
 @end
 
 @implementation FakedDragInfo
@@ skipping 953 matching lines @@
   [folder deleteBookmark:folder];
   EXPECT_FALSE([folder folderController]);
 }
 
 // TODO(jrg): draggingEntered: and draggingExited: trigger timers so
 // they are hard to test.  Factor out "fire timers" into routines
 // which can be overridden to fire immediately to make behavior
 // confirmable.
 // There is a similar problem with mouseEnteredButton: and
 // mouseExitedButton:.