[Contextual Search] Fetch and display thumbnails returned in resolution response

chrome/browser/android/compositor/scene_layer/contextual_search_scene_layer.h

Index: chrome/browser/android/compositor/scene_layer/contextual_search_scene_layer.h
diff --git a/chrome/browser/android/compositor/scene_layer/contextual_search_scene_layer.h b/chrome/browser/android/compositor/scene_layer/contextual_search_scene_layer.h
index 22cefaab4f3de08e8d8251980ee0f36c8964bbb3..271ed6897165c94d7a937ca0a57e06441fa3770b 100644
--- a/chrome/browser/android/compositor/scene_layer/contextual_search_scene_layer.h
+++ b/chrome/browser/android/compositor/scene_layer/contextual_search_scene_layer.h
@@ -13,6 +13,7 @@
 #include "base/android/scoped_java_ref.h"
 #include "base/macros.h"
 #include "chrome/browser/android/compositor/scene_layer/scene_layer.h"
+#include "chrome/browser/bitmap_fetcher/bitmap_fetcher.h"
 
 namespace cc {
 class Layer;
@@ -24,7 +25,8 @@ namespace android {
 
 class ContextualSearchLayer;
 
-class ContextualSearchSceneLayer : public SceneLayer {
+class ContextualSearchSceneLayer : public SceneLayer,
+    public chrome::BitmapFetcherDelegate {
  public:
   ContextualSearchSceneLayer(JNIEnv* env, jobject jobj);
   ~ContextualSearchSceneLayer() override;
@@ -80,13 +82,22 @@ class ContextualSearchSceneLayer : public SceneLayer {
       jfloat search_bar_shadow_opacity,
       jboolean search_provider_icon_sprite_visible,
       jfloat search_provider_icon_sprite_completion_percentage,
+      jboolean thumbnail_visible,
+      jint thumbnail_size,
+      jstring j_thumbnail_url,
       jfloat arrow_icon_opacity,
       jfloat arrow_icon_rotation,
       jfloat close_icon_opacity,
       jboolean progress_bar_visible,
       jfloat progress_bar_height,
       jfloat progress_bar_opacity,
-      jint progress_bar_completion);
+      jint progress_bar_completion,
+      jobject j_profile);
+
+  // Inherited from BitmapFetcherDelegate
+  void OnFetchComplete(
+      const GURL& url,
+      const SkBitmap* bitmap) override;
 
   void SetContentTree(
       JNIEnv* env,
@@ -98,11 +109,16 @@ class ContextualSearchSceneLayer : public SceneLayer {
       const base::android::JavaParamRef<jobject>& jobj);
 
  private:
+  JNIEnv* env_;
+  base::android::ScopedJavaGlobalRef<jobject> object_;
   float base_page_brightness_;
+  std::string thumbnail_url_;
 
   scoped_refptr<ContextualSearchLayer> contextual_search_layer_;
   scoped_refptr<cc::Layer> content_container_;
 
+  void FetchThumbnail(jobject j_profile);

[Donn Denman, 2016/09/08 18:13:39]

nit: move up to top of private?  IIRC we normally declare methods before data.

[Theresa, 2016/09/08 20:18:23]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(ContextualSearchSceneLayer);
 };