[Eraser] First pass at removing the notification center panel.

chrome/browser/ui/cocoa/notifications/message_center_tray_bridge.h

 // Copyright (c) 2013 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.
 
 #ifndef CHROME_BROWSER_UI_COCOA_NOTIFICATIONS_MESSAGE_CENTER_TRAY_BRIDGE_H_
 #define CHROME_BROWSER_UI_COCOA_NOTIFICATIONS_MESSAGE_CENTER_TRAY_BRIDGE_H_
 
 #import <AppKit/AppKit.h>
 
 #include "base/basictypes.h"
 #include "base/mac/scoped_nsobject.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "base/prefs/pref_member.h"
 #include "ui/message_center/message_center.h"
 #include "ui/message_center/message_center_tray_delegate.h"
 
 @class MCPopupCollection;
-@class MCStatusItemView;
-@class MCTrayController;
 
 namespace message_center {
 class MessageCenter;
 class MessageCenterTray;
 }
 
 // MessageCenterTrayBridge is the owner of all the Cocoa UI objects for the
 // message_center. It bridges C++ notifications from the MessageCenterTray to
 // the various UI objects.
 class MessageCenterTrayBridge :
@@ skipping 11 matching lines @@
   void HideMessageCenter() override;
   bool ShowNotifierSettings() override;
   bool IsContextMenuEnabled() const override;
   message_center::MessageCenterTray* GetMessageCenterTray() override;
 
   message_center::MessageCenter* message_center() { return message_center_; }
 
  private:
   friend class MessageCenterTrayBridgeTest;
 
-  // Updates the unread count on the status item.
-  void UpdateStatusItem();
-
-  // Opens the message center tray.
-  void OpenTrayWindow();
-
-  // Returns whether the status item allowed based on the preference value.
-  bool CanShowStatusItem() const;
-
-  // Returns whether the status item should be shown if allowed.
-  bool NeedsStatusItem() const;
-
-  // Constructs the status item view and registers its callback.
-  void ShowStatusItem();
-
-  // Destroys the status item view.
-  void HideStatusItem();
-
-  // Called after returning to the event loop from the
-  // OnMessageCenterTrayChanged event to prevent multiple updates from being
-  // rendered as the message center is modified.
-  void HandleMessageCenterTrayChanged();
-
-  // Notifies that the user has changed the show menubar item preference.
-  void OnShowStatusItemChanged();
-
   // The global, singleton message center model object. Weak.
   message_center::MessageCenter* message_center_;
 
   // C++ controller for the notification tray UI.
   scoped_ptr<message_center::MessageCenterTray> tray_;

[Robert Sesek, 2015/09/15 22:07:08]

I assume we still need |tray_| per the CL description, in that the interfaces
need to be refactored and then this can be removed?

[dewittj, 2015/09/15 22:52:22]

I was assuming so, but now that I look it's only used in one place so I'll just
modify that place and then we can remove this member var entirely.

 
-  // Obj-C window controller for the notification tray.
-  base::scoped_nsobject<MCTrayController> tray_controller_;
-
-  // View that is displayed on the system menu bar item.
-  base::scoped_nsobject<MCStatusItemView> status_item_view_;
-
   // Obj-C controller for the on-screen popup notifications.
   base::scoped_nsobject<MCPopupCollection> popup_collection_;
 
-  // Used to ensure only one status item update happens in a given call stack.
-  bool status_item_update_pending_;
-
-  // A PrefMember that calls OnShowStatusItemChanged when the pref is updated
-  // by the user's selection in the main menu.
-  BooleanPrefMember show_status_item_;
-
-  // Weak pointer factory to posts tasks to self.
-  base::WeakPtrFactory<MessageCenterTrayBridge> weak_ptr_factory_;
-
   DISALLOW_COPY_AND_ASSIGN(MessageCenterTrayBridge);
 };
 
 #endif  // CHROME_BROWSER_UI_COCOA_NOTIFICATIONS_MESSAGE_CENTER_TRAY_BRIDGE_H_