Disable use of ShowBubbleView() on Android.

chrome/browser/ui/global_error/global_error.cc

Index: chrome/browser/ui/global_error/global_error.cc
diff --git a/chrome/browser/ui/global_error/global_error.cc b/chrome/browser/ui/global_error/global_error.cc
index 9bfe3a9cd2af6abb1c7437817d2fc730f48d7fc9..1f1b06faa24842b25589db1e9d70cfa5c2866805 100644
--- a/chrome/browser/ui/global_error/global_error.cc
+++ b/chrome/browser/ui/global_error/global_error.cc
@@ -31,8 +31,12 @@ bool GlobalError::HasShownBubbleView() {
 
 void GlobalError::ShowBubbleView(Browser* browser) {
   has_shown_bubble_view_ = true;
+#if defined(OS_ANDROID)

[Peter Kasting, 2012/07/09 17:41:43]

Is it correct to leave setting |has_shown_bubble_view_| un-guarded?  I'm not
familiar with this code.

[Philippe, 2012/07/09 18:10:30]

I'm not familiar with this code either :/ My goal was really to make unit_tests
link successfully. These are the changes we have internally. It seems that we
don't use GlobalError on Android although we compile it.

[Yaron, 2012/07/09 23:23:22]

So the upshot is that right now we're silently swallowing sync errors unless you
visit about:sync (all other sources of bubble errors can't occur on Android). I
think it's fine to leave this as-is. However, please file a crbug (and reference
here) to track implementing this feature.

[Philippe, 2012/07/10 11:21:55]

Done.

+  NOTIMPLEMENTED() << "Chrome for Android doesn't support global errors";
+#else
   bubble_view_ =
       GlobalErrorBubbleViewBase::ShowBubbleView(browser, AsWeakPtr());
+#endif
 }
 
 GlobalErrorBubbleViewBase* GlobalError::GetBubbleView() {