Temp fix for blank view. Somehow the third calls to AddIconToLayout() cause this strange bug. Tried…

chrome/browser/ui/views/translate/translate_bubble_view.cc

Index: chrome/browser/ui/views/translate/translate_bubble_view.cc
diff --git a/chrome/browser/ui/views/translate/translate_bubble_view.cc b/chrome/browser/ui/views/translate/translate_bubble_view.cc
index 6b4dba941d56d103489d88e2e49abdd103dbe5a1..12753f9e0743694c252f6fb23ba710f6ffd4e273 100644
--- a/chrome/browser/ui/views/translate/translate_bubble_view.cc
+++ b/chrome/browser/ui/views/translate/translate_bubble_view.cc
@@ -85,6 +85,10 @@ views::Link* CreateLink(views::LinkListener* listener,
   return CreateLink(listener, l10n_util::GetStringUTF16(resource_id), id);
 }
 
+// Somehow the third views calling AddIconToLayout show blank as in

[msw, 2016/05/10 20:43:21]

This comment isn't very good. What does "third views" mean? Use the more common
'crbug.com' url, s/hook/hood/, who is 'we', and what do you mean by caching in
skia (caching what?) and what global variable did you try?

Instead, consider saying something like this:
// TODO(ftang) Restore icons in CreateViewAfterTranslate and CreateViewError 
// without causing layout issues; see http://crbug.com/610351

Or instead put two comments like this in the respective functions:
// TODO(ftang) Restore icon without causing layout defects: crbug.com/610351

[ftang, 2016/05/10 21:17:26]

Any third view (regardless which one) show the icon.
Somehow there are no problem for the first two views (regardless
which two) to show the icon but the third view (regardless which 
one) add the icon will cause it to become blank.

[ftang, 2016/05/10 21:17:26]

Done.

+// https://bugs.chromium.org/p/chromium/issues/detail?id=610351
+// We cannot figure out what is going on under the hook. Caching the Skia in
+// a global variable didn't help.
 void AddIconToLayout(views::GridLayout* layout) {
   views::ImageView* icon = new views::ImageView();
   ui::ResourceBundle& bundle = ui::ResourceBundle::GetSharedInstance();
@@ -703,11 +707,6 @@ views::View* TranslateBubbleView::CreateViewAfterTranslate() {
   };
 
   views::ColumnSet* cs = layout->AddColumnSet(COLUMN_SET_ID_MESSAGE);
-  if (Use2016Q2UI()) {
-    cs->AddColumn(GridLayout::LEADING, GridLayout::CENTER, 0,
-                  GridLayout::USE_PREF, 0, 0);
-    cs->AddPaddingColumn(0, views::kRelatedButtonHSpacing);
-  }
   cs->AddColumn(GridLayout::LEADING, GridLayout::CENTER, 0,
                 GridLayout::USE_PREF, 0, 0);
   cs->AddPaddingColumn(0, views::kRelatedButtonHSpacing);
@@ -721,8 +720,6 @@ views::View* TranslateBubbleView::CreateViewAfterTranslate() {
                 GridLayout::USE_PREF, 0, 0);
 
   layout->StartRow(0, COLUMN_SET_ID_MESSAGE);
-  if (Use2016Q2UI())
-    AddIconToLayout(layout);
   layout->AddView(label);
   layout->AddView(
       CreateLink(this, IDS_TRANSLATE_BUBBLE_ADVANCED, LINK_ID_ADVANCED));
@@ -753,11 +750,6 @@ views::View* TranslateBubbleView::CreateViewError() {
   };
 
   views::ColumnSet* cs = layout->AddColumnSet(COLUMN_SET_ID_MESSAGE);
-  if (Use2016Q2UI()) {
-    cs->AddColumn(GridLayout::LEADING, GridLayout::CENTER, 0,
-                  GridLayout::USE_PREF, 0, 0);
-    cs->AddPaddingColumn(0, views::kRelatedButtonHSpacing);
-  }
   cs->AddColumn(GridLayout::LEADING, GridLayout::CENTER, 0,
                 GridLayout::USE_PREF, 0, 0);
   cs->AddPaddingColumn(0, views::kRelatedButtonHSpacing);
@@ -771,8 +763,6 @@ views::View* TranslateBubbleView::CreateViewError() {
                 GridLayout::USE_PREF, 0, 0);
 
   layout->StartRow(0, COLUMN_SET_ID_MESSAGE);
-  if (Use2016Q2UI())
-    AddIconToLayout(layout);
   layout->AddView(label);
   layout->AddView(
       CreateLink(this, IDS_TRANSLATE_BUBBLE_ADVANCED, LINK_ID_ADVANCED));