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

chrome/browser/cocoa/download_item_controller.h

Index: chrome/browser/cocoa/download_item_controller.h
===================================================================
--- chrome/browser/cocoa/download_item_controller.h	(revision 0)
+++ chrome/browser/cocoa/download_item_controller.h	(revision 0)
@@ -0,0 +1,39 @@
+// 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 "base/scoped_ptr.h"
+
+class BaseDownloadItemModel;
+class DownloadItemMac;
+class DownloadShelfContextMenuMac;
+
+// A controller class that manages one download item.
+
+@interface DownloadItemController : NSViewController {
+ @private
+  IBOutlet NSPopUpButton* popupButton_;
+
+  IBOutlet NSMenu* activeDownloadMenu_;
+  IBOutlet NSMenu* completeDownloadMenu_;
+
+
+  scoped_ptr<DownloadItemMac> bridge_;
+  scoped_ptr<DownloadShelfContextMenuMac> menu_bridge_;

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

objective-C naming (menuBridge) in objective-C classes.

+};
+
+// Takes ownership of |download_model|.
+- (id)initWithFrame:(NSRect)frameRect
+           download:(BaseDownloadItemModel*)download_model;

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

I'd name this |-initWithFrame:model:|, but that's just me.

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

Fix param name to obj-C naming. Elsewhere in file too.

+
+- (void)setStateFromDownload:(BaseDownloadItemModel*)download_model;
+
+// context menu handlers
+- (IBAction)handleOpen:(id)sender;
+- (IBAction)handleAlwaysOpen:(id)sender;
+- (IBAction)handleReveal:(id)sender;
+- (IBAction)handleCancel:(id)sender;
+
+@end
Property changes on: chrome/browser/cocoa/download_item_controller.h
___________________________________________________________________
Added: svn:eol-style
   + LF