Report initial CustomTab navigation with client package name to DataUseTabUIManager

chrome/browser/android/data_usage/data_use_tab_ui_manager_android.cc

Index: chrome/browser/android/data_usage/data_use_tab_ui_manager_android.cc
diff --git a/chrome/browser/android/data_usage/data_use_tab_ui_manager_android.cc b/chrome/browser/android/data_usage/data_use_tab_ui_manager_android.cc
index 79051dea4f7c7c707214021de3e076ccb4fa1c81..7266ea19c4ec0189be1e7d4a1d81e92ed353d930 100644
--- a/chrome/browser/android/data_usage/data_use_tab_ui_manager_android.cc
+++ b/chrome/browser/android/data_usage/data_use_tab_ui_manager_android.cc
@@ -30,6 +30,16 @@ jboolean IsDataUseSessionExiting(JNIEnv* env,
   return false;
 }
 
+// static
+void CustomTabIsLoadingURL(JNIEnv* env,

[bengr, 2015/11/05 23:50:24]

Coordinate with tbansal on the name, but it should probably be something like
OnCustomTabInitialNavigation(). I'd probably add the url too.

[megjablon, 2015/11/06 01:30:43]

Renamed. @tbansal you good with this name? What do we need the url for? 

[tbansal1, 2015/11/06 01:36:22]

Name sgtm. url is needed because combination of package name and 
url will determine whether the tab is covered by the matching rules
or not. That will in turn decide which UI element to show.

+                           const JavaParamRef<jclass>& clazz,
+                           jint tab_id,
+                           const JavaParamRef<jstring>& packageName,
+                           const JavaParamRef<jobject>& jprofile) {
+  // TODO(megjablon): Get the DataUseTabUIManager which is a keyed service and
+  // tell it about the custom tab package.
+}
+
 bool RegisterDataUseTabUIManager(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }