Add ink drop ripple to status tray

ash/common/system/tray/system_tray.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.

[bruthig, 2016/11/17 06:31:20]

Where is the ink drop placed in RTL mode?  Does that apply to all UI elements
that have an ink drop and center it on the click point?

[mohsen, 2016/11/17 20:10:12]

It's not related to the click point. It's related to different left and right
insets we might have for the ink drop. BTW, the issue is now fixed.

 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef ASH_COMMON_SYSTEM_TRAY_SYSTEM_TRAY_H_
 #define ASH_COMMON_SYSTEM_TRAY_SYSTEM_TRAY_H_
 
 #include <map>
 #include <memory>
 #include <vector>
 
@@ skipping 153 matching lines @@
 
   TrayCast* GetTrayCastForTesting() const;
   TrayDate* GetTrayDateForTesting() const;
   TraySystemInfo* GetTraySystemInfoForTesting() const;
   TrayTiles* GetTrayTilesForTesting() const;
   TrayUpdate* GetTrayUpdateForTesting() const;
 
   // Activates the system tray bubble.
   void ActivateBubble();
 
+ protected:
+  // Overridden from TrayBackgroundView.
+  gfx::Insets GetBackgroundInsets() const override;
+
  private:
   class ActivationObserver;
 
   // Closes the bubble. Used to bind as a KeyEventWatcher::KeyEventCallback.
   void CloseBubble(const ui::KeyEvent& key_event);
 
   // Activates the bubble and starts key navigation with the |key_event|.
   void ActivateAndStartNavigation(const ui::KeyEvent& key_event);
 
   // Creates the key event watcher. See |ShowItems()| for why key events are
@@ skipping 72 matching lines @@
   // Set to true when system notifications should be hidden (e.g. web
   // notification bubble is visible).
   bool hide_notifications_;
 
   // This is true when the displayed system tray menu is a full tray menu,
   // otherwise a single line item menu like the volume slider is shown.
   // Note that the value is only valid when |system_bubble_| is true.
   bool full_system_tray_menu_;
 
   // These objects are not owned by this class.
+  TrayUser* tray_user_active_;

[bruthig, 2016/11/17 06:31:20]

Consider renaming as |active_tray_user_|.

[mohsen, 2016/11/17 20:10:12]

Not needed anymore. Removed.

   TrayAccessibility* tray_accessibility_;
   TrayAudio* tray_audio_;  // May be null.
   TrayCast* tray_cast_;
   TrayDate* tray_date_;    // null for material design.
   TrayTiles* tray_tiles_;  // only used in material design.
   TraySystemInfo* tray_system_info_;  // only used in material design.
   TrayUpdate* tray_update_;
 
   // A reference to the Screen share and capture item.
   ScreenTrayItem* screen_capture_tray_item_;  // not owned
   ScreenTrayItem* screen_share_tray_item_;    // not owned
 
   // TODO(oshima): Remove this when crbug.com/651242 is fixed.
   bool activating_ = false;
 
   std::unique_ptr<KeyEventWatcher> key_event_watcher_;
 
   std::unique_ptr<ActivationObserver> activation_observer_;
 
   DISALLOW_COPY_AND_ASSIGN(SystemTray);
 };
 
 }  // namespace ash
 
 #endif  // ASH_COMMON_SYSTEM_TRAY_SYSTEM_TRAY_H_