Move NewTabPage.LogoShownTime metrics recording into LogoBridge

chrome/browser/android/logo_bridge.h

 // Copyright 2014 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.
 
 #ifndef CHROME_BROWSER_ANDROID_LOGO_BRIDGE_H_
 #define CHROME_BROWSER_ANDROID_LOGO_BRIDGE_H_
 
 #include <jni.h>
 
 #include <memory>
@@ skipping 17 matching lines @@
 
 namespace image_fetcher {
 class ImageFetcher;
 struct RequestMetadata;
 }  // namespace image_fetcher
 
 // The C++ counterpart to LogoBridge.java. Enables Java code to access the
 // default search provider's logo.
 class LogoBridge : public doodle::DoodleService::Observer {
  public:
+  // Stores its creation time and writes it to a UMA histogram. Each instance
+  // writes the metric once or never.
+  class LoadTimeMetricRecorder {
+   public:
+    // Writes the diff between |Now| and the recorders creation time to a UMA
+    // metric. If the recorder was aborted/finished before, nothing happens.
+    virtual void Finish() = 0;
+
+    // The recorder instance immediately stops to record.
+    virtual void Cancel() = 0;
+  };
+
   explicit LogoBridge(jobject j_profile);
   void Destroy(JNIEnv* env, const base::android::JavaParamRef<jobject>& obj);
 
   // Gets the current non-animated logo (downloading it if necessary) and passes
   // it to the observer.
   void GetCurrentLogo(
       JNIEnv* env,
       const base::android::JavaParamRef<jobject>& obj,
       const base::android::JavaParamRef<jobject>& j_logo_observer);
 
   // Downloads the animated logo from the given URL and returns it to the
   // callback. Does not support multiple concurrent requests: A second request
   // for the same URL will be ignored; a request for a different URL will cancel
   // any ongoing request. If downloading fails, the callback is not called.
   void GetAnimatedLogo(JNIEnv* env,
                        const base::android::JavaParamRef<jobject>& obj,
                        const base::android::JavaParamRef<jobject>& j_callback,
                        const base::android::JavaParamRef<jstring>& j_url);
 
  private:
   class AnimatedLogoFetcher;
 
   virtual ~LogoBridge();
 
   // doodle::DoodleService::Observer implementation.
   void OnDoodleConfigUpdated(
       const base::Optional<doodle::DoodleConfig>& maybe_doodle_config) override;
 
-  void DoodleConfigReceived(
+  // Takes a config and loads the image for it. Returns true if the image is
+  // loading.
+  bool DoodleConfigReceived(
       const base::Optional<doodle::DoodleConfig>& maybe_doodle_config,
       bool from_cache);
 
   void DoodleImageFetched(bool config_from_cache,
                           const GURL& on_click_url,
                           const std::string& alt_text,
                           const GURL& animated_image_url,
                           const std::string& image_fetch_id,
                           const gfx::Image& image,
                           const image_fetcher::RequestMetadata& metadata);
 
+  // Returns the currently running LoadTimeRecorder. If there is none, a null
+  // object is returned.
+  LoadTimeMetricRecorder* load_time_recorder();
+
   // Only valid if UseNewDoodleApi is disabled.
   LogoService* logo_service_;
 
   // Only valid if UseNewDoodleApi is enabled.
   doodle::DoodleService* doodle_service_;
   std::unique_ptr<image_fetcher::ImageFetcher> image_fetcher_;
   base::android::ScopedJavaGlobalRef<jobject> j_logo_observer_;
   ScopedObserver<doodle::DoodleService, doodle::DoodleService::Observer>
       doodle_observer_;
 
   std::unique_ptr<AnimatedLogoFetcher> animated_logo_fetcher_;
 
+  std::unique_ptr<LoadTimeMetricRecorder> load_time_recorder_;
+
   base::WeakPtrFactory<LogoBridge> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(LogoBridge);
 };
 
 bool RegisterLogoBridge(JNIEnv* env);
 
 #endif  // CHROME_BROWSER_ANDROID_LOGO_BRIDGE_H_