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

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

 // Copyright 2015 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.
 
 #include "chrome/browser/android/compositor/scene_layer/contextual_search_scene_layer.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
+#include "base/android/jni_string.h"
 #include "cc/layers/solid_color_layer.h"
 #include "chrome/browser/android/compositor/layer/contextual_search_layer.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_android.h"
 #include "content/public/browser/android/compositor.h"
 #include "content/public/browser/web_contents.h"
 #include "jni/ContextualSearchSceneLayer_jni.h"
+#include "net/base/load_flags.h"
+#include "net/url_request/url_request_context_getter.h"
 #include "ui/android/resources/resource_manager_impl.h"
 #include "ui/android/view_android.h"
 #include "ui/gfx/android/java_bitmap.h"
 
 using base::android::JavaParamRef;
 
 namespace chrome {
 namespace android {
 
 ContextualSearchSceneLayer::ContextualSearchSceneLayer(JNIEnv* env,
                                                        jobject jobj)
     : SceneLayer(env, jobj),
+      env_(env),
+      object_(env, jobj),
       base_page_brightness_(1.0f),
       content_container_(cc::Layer::Create()) {
   // Responsible for moving the base page without modifying the layer itself.
   content_container_->SetIsDrawable(true);
   content_container_->SetPosition(gfx::PointF(0.0f, 0.0f));
   layer()->AddChild(content_container_);
 }
 
 void ContextualSearchSceneLayer::CreateContextualSearchLayer(
     JNIEnv* env,
@@ skipping 51 matching lines @@
     jfloat search_context_opacity,
     jfloat search_term_opacity,
     jfloat search_caption_animation_percentage,
     jboolean search_caption_visible,
     jboolean search_bar_border_visible,
     jfloat search_bar_border_height,
     jboolean search_bar_shadow_visible,
     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) {
+
+  // Load the thumbnail if necessary.
+  std::string thumbnail_url =
+      base::android::ConvertJavaStringToUTF8(env, j_thumbnail_url);
+  if (thumbnail_url.compare(thumbnail_url_) != 0) {
+    thumbnail_url_ = thumbnail_url;
+    FetchThumbnail(j_profile);
+  }
+
   // NOTE(pedrosimonetti): The WebContents might not exist at this time if
   // the Contextual Search Result has not been requested yet. In this case,
   // we'll pass NULL to Contextual Search's Layer Tree.
   content::WebContents* web_contents =
       content::WebContents::FromJavaWebContents(jweb_contents);
 
   scoped_refptr<cc::Layer> content_layer =
       web_contents ? web_contents->GetNativeView()->GetLayer() : nullptr;
 
   // Fade the base page out.
@@ skipping 45 matching lines @@
       search_context_opacity,
       search_term_opacity,
       search_caption_animation_percentage,
       search_caption_visible,
       search_bar_border_visible,
       search_bar_border_height,
       search_bar_shadow_visible,
       search_bar_shadow_opacity,
       search_provider_icon_sprite_visible,
       search_provider_icon_sprite_completion_percentage,
+      thumbnail_visible,
+      thumbnail_size,
       arrow_icon_opacity,
       arrow_icon_rotation,
       close_icon_opacity,
       progress_bar_visible,
       progress_bar_height,
       progress_bar_opacity,
       progress_bar_completion);
 
   // Make the layer visible if it is not already.
   contextual_search_layer_->layer()->SetHideLayerAndSubtree(false);
 }
 
+void ContextualSearchSceneLayer::FetchThumbnail(jobject j_profile) {
+  if (thumbnail_url_.compare("") == 0) return;
+
+  GURL* gurl = new GURL(thumbnail_url_);
+  Profile* profile = ProfileAndroid::FromProfileAndroid(j_profile);
+  chrome::BitmapFetcher* fetcher = new chrome::BitmapFetcher(*gurl, this);

[mdjones, 2016/09/08 17:14:21]

Does BitmapFetcher delete itself when it is done?

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

I don't think so. Add an explicit call to the destructor below.

+  fetcher->Init(
+      profile->GetRequestContext(),
+      std::string(),
+      net::URLRequest::CLEAR_REFERRER_ON_TRANSITION_FROM_SECURE_TO_INSECURE,
+      net::LOAD_NORMAL);
+  fetcher->Start();
+}
+
+void ContextualSearchSceneLayer::OnFetchComplete(const GURL& url,
+                                                 const SkBitmap* bitmap) {
+  bool success = bitmap != NULL && !bitmap->drawsNothing();
+  Java_ContextualSearchSceneLayer_onThumbnailFetched(env_,
+                                                     object_.obj(),
+                                                     success);
+
+  contextual_search_layer_->SetThumbnail(bitmap);

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

Should this line be conditional on success?

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

Yes, done.

+}
+
 void ContextualSearchSceneLayer::SetContentTree(
     JNIEnv* env,
     const JavaParamRef<jobject>& jobj,
     const JavaParamRef<jobject>& jcontent_tree) {
   SceneLayer* content_tree = FromJavaObject(env, jcontent_tree);
   if (!content_tree || !content_tree->layer()) return;
 
   if (!content_tree->layer()->parent()
       || (content_tree->layer()->parent()->id() != content_container_->id())) {
     content_container_->AddChild(content_tree->layer());
@@ skipping 20 matching lines @@
       new ContextualSearchSceneLayer(env, jobj);
   return reinterpret_cast<intptr_t>(tree_provider);
 }
 
 bool RegisterContextualSearchSceneLayer(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace android
 }  // namespace chrome