[Home] Record some user actions for the Chrome Home BottomSheet

chrome/android/java/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheetContentController.java

Index: chrome/android/java/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheetContentController.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheetContentController.java b/chrome/android/java/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheetContentController.java
index d8222e8a8557e8bc14d7d45c1782ac750b6f28f2..ea8f1cb514c190ae0bb76235a5be47076a52d5f4 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheetContentController.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheetContentController.java
@@ -123,6 +123,12 @@ public class BottomSheetContentController extends BottomNavigationView
     @Override
     public void onSheetOffsetChanged(float heightFraction) {}
 
+    @Override
+    public void onSheetStateChanged(int newState) {}
+
+    @Override
+    public void onSheetContentChanged(BottomSheetContent newContent) {}

[Theresa, 2017/03/17 16:07:12]

I'm considering making a new class variable for the BottomSheetObserver that
extends EmptyBottomSheetObserver to avoid the need for these empty methods.
Thoughts? Should I do the same for BottomToolbarPhone?

[mdjones, 2017/03/17 18:17:26]

Yeah, I think that interface is sufficiently large to support doing this.

+
     // TODO(twellington): remove this once the support library is updated to allow disabling
     //                    shifting mode or determines shifting mode based on the width of the
     //                    child views.