Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(428)

Side by Side Diff: chrome/browser/ui/cocoa/download/download_shelf_controller.h

Issue 1221173003: [Mac] Inform reference counted objects that hold a weak Browser* when the Browser is being destroye… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Keep -[BookmarBarController dealloc] instead of changing all the tests. Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #import <Cocoa/Cocoa.h> 5 #import <Cocoa/Cocoa.h>
6 6
7 #include "base/mac/scoped_nsobject.h" 7 #include "base/mac/scoped_nsobject.h"
8 #include "base/memory/scoped_ptr.h" 8 #include "base/memory/scoped_ptr.h"
9 #import "chrome/browser/ui/cocoa/view_resizer.h" 9 #import "chrome/browser/ui/cocoa/view_resizer.h"
10 #include "ui/base/cocoa/tracking_area.h" 10 #include "ui/base/cocoa/tracking_area.h"
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
79 // Delegate that handles resizing our view. 79 // Delegate that handles resizing our view.
80 id<ViewResizer> resizeDelegate_; 80 id<ViewResizer> resizeDelegate_;
81 81
82 // Used for loading pages. 82 // Used for loading pages.
83 content::PageNavigator* navigator_; 83 content::PageNavigator* navigator_;
84 }; 84 };
85 85
86 - (id)initWithBrowser:(Browser*)browser 86 - (id)initWithBrowser:(Browser*)browser
87 resizeDelegate:(id<ViewResizer>)resizeDelegate; 87 resizeDelegate:(id<ViewResizer>)resizeDelegate;
88 88
89 // Called by BrowserWindowController when the Browser is being destroyed.
90 - (void)browserWillBeDestroyed;
91
89 // Run when the user clicks the 'Show All' button. 92 // Run when the user clicks the 'Show All' button.
90 - (IBAction)showDownloadsTab:(id)sender; 93 - (IBAction)showDownloadsTab:(id)sender;
91 94
92 // Run when the user clicks the close button on the right side of the shelf. 95 // Run when the user clicks the close button on the right side of the shelf.
93 - (IBAction)handleClose:(id)sender; 96 - (IBAction)handleClose:(id)sender;
94 97
95 // Shows or hides the download shelf based on the value of |show|. 98 // Shows or hides the download shelf based on the value of |show|.
96 // |isUserAction| should be YES if the operation is being triggered based on a 99 // |isUserAction| should be YES if the operation is being triggered based on a
97 // user action (currently only relevant when hiding the shelf). 100 // user action (currently only relevant when hiding the shelf).
98 // Note: This is intended to be invoked from DownloadShelfMac. If invoked 101 // Note: This is intended to be invoked from DownloadShelfMac. If invoked
(...skipping 14 matching lines...) Expand all
113 116
114 // Similar to addDownloadItem above, but adds a DownloadItemController. 117 // Similar to addDownloadItem above, but adds a DownloadItemController.
115 - (void)add:(DownloadItemController*)download; 118 - (void)add:(DownloadItemController*)download;
116 119
117 // Remove a download, possibly via clearing browser data. 120 // Remove a download, possibly via clearing browser data.
118 - (void)remove:(DownloadItemController*)download; 121 - (void)remove:(DownloadItemController*)download;
119 122
120 // Called by individual item controllers when their downloads are opened. 123 // Called by individual item controllers when their downloads are opened.
121 - (void)downloadWasOpened:(DownloadItemController*)download; 124 - (void)downloadWasOpened:(DownloadItemController*)download;
122 125
123 // Notification that the download shelf is going to be destroyed and should
124 // release the downloads.
125 - (void)exiting;
126
127 // Return the height of the download shelf. 126 // Return the height of the download shelf.
128 - (float)height; 127 - (float)height;
129 128
130 // Re-layouts all download items based on their current state. 129 // Re-layouts all download items based on their current state.
131 - (void)layoutItems; 130 - (void)layoutItems;
132 131
133 @end 132 @end
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698