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

chrome/android/java/src/org/chromium/chrome/browser/bookmarks/BookmarkSheetContent.java

 // Copyright 2017 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.
 
 package org.chromium.chrome.browser.bookmarks;
 
 import android.support.v7.widget.Toolbar;
 import android.view.View;
 
 import org.chromium.chrome.browser.ChromeActivity;
 import org.chromium.chrome.browser.UrlConstants;
 import org.chromium.chrome.browser.toolbar.BottomToolbarPhone;
 import org.chromium.chrome.browser.widget.bottomsheet.BottomSheet.BottomSheetContent;
+import org.chromium.chrome.browser.widget.bottomsheet.BottomSheetContentController;
 
 /**
  * A {@link BottomSheetContent} holding a {@link BookmarkManager} for display in the BottomSheet.
  */
 public class BookmarkSheetContent implements BottomSheetContent {
     private final View mContentView;
     private final Toolbar mToolbarView;
     private BookmarkManager mBookmarkManager;
 
     /**
@@ skipping 21 matching lines @@
     @Override
     public int getVerticalScrollOffset() {
         return mBookmarkManager.getVerticalScrollOffset();
     }
 
     @Override
     public void destroy() {
         mBookmarkManager.destroy();
         mBookmarkManager = null;
     }
+
+    @Override
+    public int getType() {

[gone, 2017/03/20 20:29:31]

Does it work if you try to annotate these with @ContentType?

[Theresa, 2017/03/20 20:59:12]

I added an annotation to the superclass method. Will that suffice or do you want
to see it on every subclass method?

[gone, 2017/03/20 21:04:52]

super's probably cool.  Hard to gauge where it should go, given that this thing
is Overriding something.  I'd ask if Android Studio complains, but I'm not sure
anyone's really using that at the moment.

[Theresa, 2017/03/20 21:22:52]

With the superclass annotation, Android Studio complains if the subclass
implementation returns an int rather than a ContentType. Eclipse doesn't seem to
care regardless of annotations.

+        return BottomSheetContentController.TYPE_BOOKMARKS;
+    }
 }