Restrict sending GWS ID only to signed in users

chrome/browser/android/data_usage/external_data_use_observer_bridge.cc

Index: chrome/browser/android/data_usage/external_data_use_observer_bridge.cc
diff --git a/chrome/browser/android/data_usage/external_data_use_observer_bridge.cc b/chrome/browser/android/data_usage/external_data_use_observer_bridge.cc
index 01f57f4fc9366e3da0a15c7a53a07ad7c32434c3..a8e4171950f28b8d471c39578d76a48a5adee09f 100644
--- a/chrome/browser/android/data_usage/external_data_use_observer_bridge.cc
+++ b/chrome/browser/android/data_usage/external_data_use_observer_bridge.cc
@@ -17,6 +17,10 @@
 #include "chrome/browser/android/data_usage/data_use_tab_model.h"
 #include "chrome/browser/android/data_usage/external_data_use_observer.h"
 #include "chrome/browser/metrics/chrome_metrics_service_accessor.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/browser/signin/signin_manager_factory.h"
+#include "components/signin/core/browser/signin_manager.h"
 #include "components/variations/variations_associated_data.h"
 #include "content/public/browser/browser_thread.h"
 #include "jni/ExternalDataUseObserver_jni.h"
@@ -211,7 +215,12 @@ void ExternalDataUseObserverBridge::ShouldRegisterAsDataUseObserver(
                  external_data_use_observer_, should_register));
 
   variations::VariationID variation_id = GetGoogleVariationID();
-  if (variation_id != variations::EMPTY_ID) {
+  Profile* profile = ProfileManager::GetActiveUserProfile();

[tbansal1, 2016/11/17 18:09:31]

Comment for this method says "As such avoid
  // using it at all cost."

https://cs.chromium.org/chromium/src/chrome/browser/profiles/profile_manager....

Is there an alternative method?

[Raj, 2016/11/17 18:46:30]

Using GetLastUsedProfile() instead, which is very similar for Android chrome.

+  DCHECK(profile);
+  const SigninManager* signin_manager =
+      SigninManagerFactory::GetForProfileIfExists(profile);
+  if (variation_id != variations::EMPTY_ID && signin_manager &&
+      signin_manager->IsAuthenticated()) {
     // Set variation id for the enabled group if |should_register| is true.
     // Otherwise clear the variation id for the enabled group by setting to
     // EMPTY_ID.