Move download item to its own view and a xib, paving the way for a custom dow...

chrome/browser/cocoa/download_shelf_controller.h

 // Copyright (c) 2009 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 <vector>
 
 #include "base/scoped_ptr.h"
 
+class BaseDownloadItemModel;
 class Browser;
 @class BrowserWindowController;
+@class DownloadItemController;
 class DownloadShelf;
 @class DownloadShelfView;
 
 // A controller class that manages the download shelf for one window.
 
 @interface DownloadShelfController : NSViewController {
  @private
   IBOutlet NSTextView* showAllDownloadsLink_;
 
   // Currently these two are always the same, but they mean slightly
   // different things.  contentAreaHasOffset_ is an implementation
   // detail of download shelf visibility.
   BOOL contentAreaHasOffset_;
   BOOL barIsVisible_;
 
   scoped_ptr<DownloadShelf> bridge_;
   NSView* contentArea_;
   int shelfHeight_;
+
+  // The download items we have added to our shelf.
+  std::vector<DownloadItemController*> download_item_controllers_;

[pink (ping after 24hrs), 2009/07/06 21:45:20]

use obj-C naming. Also, make this a NSMutableArray.

 };
 
 - (id)initWithBrowser:(Browser*)browser
     contentArea:(NSView*)content;
 
 - (DownloadShelf*)bridge;
 - (BOOL)isVisible;
 
 - (IBAction)show:(id)sender;
 - (IBAction)hide:(id)sender;
 
-// TODO(thakis): this should internally build an item and get only
-// the model as parameter.
-- (void)addDownloadItem:(NSView*)view;
+- (void)addDownloadItem:(BaseDownloadItemModel*)view;

[pink (ping after 24hrs), 2009/07/06 21:45:20]

is this a model or a view? The param name should probably be changed.

 
 // Resizes the download shelf based on the state of the content area.
 - (void)resizeDownloadShelf;
 
 @end