Cleanup: Change LogMostVisitedImpression|Navigation APIs to take an enum

chrome/common/render_messages.h

 // Copyright (c) 2012 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.
 
 // Multiply-included file, no traditional include guard.
 #include <stdint.h>
 #include <string>
 #include <vector>
 
 #include "base/files/file_path.h"
@@ skipping 147 matching lines @@
   IPC_STRUCT_TRAITS_MEMBER(image_vertical_alignment)
   IPC_STRUCT_TRAITS_MEMBER(image_tiling)
   IPC_STRUCT_TRAITS_MEMBER(image_height)
   IPC_STRUCT_TRAITS_MEMBER(has_attribution)
   IPC_STRUCT_TRAITS_MEMBER(logo_alternate)
 IPC_STRUCT_TRAITS_END()
 
 IPC_ENUM_TRAITS_MAX_VALUE(NTPLoggingEventType,
                           NTP_EVENT_TYPE_LAST)
 
+IPC_ENUM_TRAITS_MAX_VALUE(NTPLoggingTileSource,
+                          NTP_TILE_SOURCE_LAST)
+
 IPC_ENUM_TRAITS_MAX_VALUE(WebApplicationInfo::MobileCapable,
                           WebApplicationInfo::MOBILE_CAPABLE_APPLE)
 
 IPC_STRUCT_TRAITS_BEGIN(WebApplicationInfo::IconInfo)
   IPC_STRUCT_TRAITS_MEMBER(url)
   IPC_STRUCT_TRAITS_MEMBER(width)
   IPC_STRUCT_TRAITS_MEMBER(height)
   IPC_STRUCT_TRAITS_MEMBER(data)
 IPC_STRUCT_TRAITS_END()
 
@@ skipping 303 matching lines @@
 IPC_MESSAGE_ROUTED3(ChromeViewHostMsg_LogEvent,
                     int /* page_seq_no */,
                     NTPLoggingEventType /* event */,
                     base::TimeDelta /* time */)
 
 // Logs an impression on one of the Most Visited tile on the InstantExtended
 // New Tab Page.
 IPC_MESSAGE_ROUTED3(ChromeViewHostMsg_LogMostVisitedImpression,
                     int /* page_seq_no */,
                     int /* position */,
-                    base::string16 /* provider */)
+                    NTPLoggingTileSource /* tile_source */)
 
 // Logs a navigation on one of the Most Visited tile on the InstantExtended
 // New Tab Page.
 IPC_MESSAGE_ROUTED3(ChromeViewHostMsg_LogMostVisitedNavigation,
                     int /* page_seq_no */,
                     int /* position */,
-                    base::string16 /* provider */)
+                    NTPLoggingTileSource /* tile_source */)
 
 // The Instant page asks whether the user syncs its history.
 IPC_MESSAGE_ROUTED1(ChromeViewHostMsg_HistorySyncCheck,
                     int /* page_seq_no */)
 
 // The Instant page asks for Chrome identity check against |identity|.
 IPC_MESSAGE_ROUTED2(ChromeViewHostMsg_ChromeIdentityCheck,
                     int /* page_seq_no */,
                     base::string16 /* identity */)
 
@@ skipping 58 matching lines @@
 
 // Record a sample string to a Rappor metric.
 IPC_MESSAGE_CONTROL2(ChromeViewHostMsg_RecordRappor,
                      std::string /* metric */,
                      std::string /* sample */)
 
 // Record a domain and registry of a url to a Rappor metric.
 IPC_MESSAGE_CONTROL2(ChromeViewHostMsg_RecordRapporURL,
                      std::string /* metric */,
                      GURL /* sample url */)