Move title creation logic to native

chrome/browser/android/compositor/layer_title_cache.cc

 // 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.
 
 #include "chrome/browser/android/compositor/layer_title_cache.h"
 
 #include <android/bitmap.h>
 
 #include <memory>
 
@@ skipping 84 matching lines @@
     const int id = iter.GetCurrentKey();
     if (id != except_id)
       layer_cache_.Remove(id);
   }
 }
 
 DecorationTitle* LayerTitleCache::GetTitleLayer(int tab_id) {
   return layer_cache_.Lookup(tab_id);
 }
 
+void LayerTitleCache::BuildTitleLayer(int tab_id) {

[David Trainor- moved to gerrit, 2016/09/09 05:29:09]

Should this logic just live in GetTitleLayer?

[mdjones, 2016/09/09 15:40:13]

Done.

+  if (layer_cache_.Lookup(tab_id))
+    return;
+
+  JNIEnv* env = base::android::AttachCurrentThread();
+  Java_LayerTitleCache_buildUpdatedTitle(env, weak_java_title_cache_.get(env),
+      tab_id);
+}
+
 void LayerTitleCache::SetResourceManager(
     ui::ResourceManager* resource_manager) {
   resource_manager_ = resource_manager;
 
   IDMap<DecorationTitle, IDMapOwnPointer>::iterator iter(&layer_cache_);
   for (; !iter.IsAtEnd(); iter.Advance()) {
     iter.GetCurrentValue()->SetResourceManager(resource_manager_);
   }
 }
 
@@ skipping 16 matching lines @@
            jint spinner_resource_id,
            jint spinner_incognito_resource_id) {
   LayerTitleCache* cache = new LayerTitleCache(
       env, obj, fade_width, favicon_start_padding, favicon_end_padding,
       spinner_resource_id, spinner_incognito_resource_id);
   return reinterpret_cast<intptr_t>(cache);
 }
 
 }  // namespace android
 }  // namespace chrome