Fix three causes of blank thumbnails [2nd land]

chrome/browser/android/compositor/tab_content_manager.cc

Index: chrome/browser/android/compositor/tab_content_manager.cc
diff --git a/chrome/browser/android/compositor/tab_content_manager.cc b/chrome/browser/android/compositor/tab_content_manager.cc
index c9146d733fedf372707a61813518dbd1633270ee..8cd25593bd2cfb1a913030554ac5d702900cf521 100644
--- a/chrome/browser/android/compositor/tab_content_manager.cc
+++ b/chrome/browser/android/compositor/tab_content_manager.cc
@@ -221,18 +221,17 @@ void TabContentManager::CacheTab(JNIEnv* env,
   content::WebContents* web_contents = tab_android->web_contents();
   DCHECK(web_contents);
 
-  if (thumbnail_cache_->CheckAndUpdateThumbnailMetaData(tab_id, url)) {
-    if (!web_contents->GetRenderViewHost() ||
-        !web_contents->GetRenderViewHost()->GetWidget() ||
-        !web_contents->GetRenderViewHost()
-             ->GetWidget()
-             ->CanCopyFromBackingStore() ||
-        pending_tab_readbacks_.find(tab_id) != pending_tab_readbacks_.end() ||
-        pending_tab_readbacks_.size() >= kMaxReadbacks) {
-      thumbnail_cache_->Remove(tab_id);
-      return;
-    }
+  if (!web_contents->GetRenderViewHost() ||
+      !web_contents->GetRenderViewHost()->GetWidget() ||
+      !web_contents->GetRenderViewHost()
+           ->GetWidget()
+           ->CanCopyFromBackingStore() ||
+      pending_tab_readbacks_.find(tab_id) != pending_tab_readbacks_.end() ||
+      pending_tab_readbacks_.size() >= kMaxReadbacks) {
+    return;
+  }
 
+  if (thumbnail_cache_->CheckAndUpdateThumbnailMetaData(tab_id, url)) {
     TabReadbackCallback readback_done_callback =
         base::Bind(&TabContentManager::PutThumbnailIntoCache,
                    weak_factory_.GetWeakPtr(), tab_id);