summaryrefslogtreecommitdiffstats
path: root/chrome/browser/cocoa/cookies_window_controller.mm
diff options
context:
space:
mode:
Diffstat (limited to 'chrome/browser/cocoa/cookies_window_controller.mm')
-rw-r--r--chrome/browser/cocoa/cookies_window_controller.mm26
1 files changed, 19 insertions, 7 deletions
diff --git a/chrome/browser/cocoa/cookies_window_controller.mm b/chrome/browser/cocoa/cookies_window_controller.mm
index cec7f71..70fddc7 100644
--- a/chrome/browser/cocoa/cookies_window_controller.mm
+++ b/chrome/browser/cocoa/cookies_window_controller.mm
@@ -327,6 +327,12 @@ bool CookiesTreeModelObserverBridge::HasCocoaModel() {
if (count != 1U) {
DCHECK_LT(count, 1U) << "User was able to select more than 1 cookie node!";
[self setRemoveButtonEnabled:NO];
+
+ // Make sure that the cookie info pane is shown when there is no selection.
+ // That's what windows does.
+ [cookieInfo_ setHidden:NO];
+ [localStorageInfo_ setHidden:YES];
+ [databaseInfo_ setHidden:YES];
return;
}
@@ -346,13 +352,15 @@ bool CookiesTreeModelObserverBridge::HasCocoaModel() {
[self setRemoveButtonEnabled:YES];
CocoaCookieTreeNodeType nodeType = [[selectedObjects lastObject] nodeType];
- if (nodeType == kCocoaCookieTreeNodeTypeLocalStorage) {
- [cookieInfo_ setHidden:YES];
- [localStorageInfo_ setHidden:NO];
- } else {
- [cookieInfo_ setHidden:NO];
- [localStorageInfo_ setHidden:YES];
- }
+ bool hideCookieInfoView = nodeType != kCocoaCookieTreeNodeTypeCookie &&
+ nodeType != kCocoaCookieTreeNodeTypeFolder;
+ bool hideLocaStorageInfoView =
+ nodeType != kCocoaCookieTreeNodeTypeLocalStorage;
+ bool hideDatabaseInfoView =
+ nodeType != kCocoaCookieTreeNodeTypeDatabaseStorage;
+ [cookieInfo_ setHidden:hideCookieInfoView];
+ [localStorageInfo_ setHidden:hideLocaStorageInfoView];
+ [databaseInfo_ setHidden:hideDatabaseInfoView];
}
#pragma mark Unit Testing
@@ -373,6 +381,10 @@ bool CookiesTreeModelObserverBridge::HasCocoaModel() {
return localStorageInfo_;
}
+- (NSView*)databaseInfoView {
+ return databaseInfo_;
+}
+
// Re-initializes the |treeModel_|, creates a new observer for it, and re-
// builds the |cocoaTreeModel_|. We use this to initialize the controller and
// to rebuild after the user clears browsing data. Because the models get