ntp_tiles: Cleanup enum names

chrome/android/java/src/org/chromium/chrome/browser/suggestions/MostVisitedSites.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.suggestions;
 
 import org.chromium.base.annotations.CalledByNative;
-import org.chromium.chrome.browser.ntp.MostVisitedTileType;
-import org.chromium.chrome.browser.ntp.NTPTileSource;
 
 /**
  * Methods to provide most recent urls, titles and thumbnails.
  */
 interface MostVisitedSites {
     /**
      * An interface for handling events in {@link MostVisitedSites}.
      */
     interface Observer {
         /**
          * This is called when the list of most visited URLs is initially available or updated.
          * Parameters guaranteed to be non-null.
          *
          * @param titles Array of most visited url page titles.
          * @param urls Array of most visited URLs, including popular URLs if
          *             available and necessary (i.e. there aren't enough most
          *             visited URLs).
          * @param whitelistIconPaths The paths to the icon image files for whitelisted tiles, empty
          *                           strings otherwise.
-         * @param sources For each tile, the {@code NTPTileSource} that generated the tile.
+         * @param sources For each tile, the {@code TileSource} that generated the tile.
          */
         @CalledByNative("Observer")
         void onMostVisitedURLsAvailable(
                 String[] titles, String[] urls, String[] whitelistIconPaths, int[] sources);
 
         /**
          * This is called when a previously uncached icon has been fetched.
          * Parameters guaranteed to be non-null.
          *
          * @param siteUrl URL of site with newly-cached icon.
@@ skipping 21 matching lines @@
     void addBlacklistedUrl(String url);
 
     /**
      * Removes a URL from the most visited URLs blacklist.
      */
     void removeBlacklistedUrl(String url);
 
     /**
      * Records metrics about an impression, including the sources (local, server, ...) and visual
      * types of the tiles that are shown.
-     * @param tileTypes An array of values from MostVisitedTileType indicating the type of each
+     * @param tileTypes An array of values from {@link TileVisualType} indicating the type of each
      *                  tile that's currently showing.
-     * @param sources An array of values from NTPTileSource indicating the source of each tile
+     * @param sources An array of values from {@link TileSource} indicating the source of each tile
      *                that's currently showing.
      * @param tileUrls An array of strings indicating the URL of each tile.
      */
     void recordPageImpression(int[] tileTypes, int[] sources, String[] tileUrls);
 
     /**
      * Records the opening of a Most Visited Item.
      * @param index The index of the item that was opened.
-     * @param type The visual type of the item as defined in {@code MostVisitedTileType}.
-     * @param source The {@code NTPTileSource} that generated this item.
+     * @param type The visual type of the item as defined in {@link TileVisualType}.
+     * @param source The {@link TileSource} that generated this item.
      */
-    void recordOpenedMostVisitedItem(int index,
-            @MostVisitedTileType.MostVisitedTileTypeEnum int type,
-            @NTPTileSource.NTPTileSourceEnum int source);
+    void recordOpenedMostVisitedItem(int index, @TileVisualType.TileVisualTypeEnum int type,

[Michael van Ouwerkerk, 2017/04/03 13:35:59]

Cool, and this will be further simplified to @TileVisualType once Nicolas's CL
lands: https://codereview.chromium.org/2792583002/

+            @TileSource.TileSourceEnum int source);
 }