Fix Android strict-mode violation in GeoLocation info bar.

chrome/browser/geolocation/geolocation_infobar_delegate.h

Index: chrome/browser/geolocation/geolocation_infobar_delegate.h
diff --git a/chrome/browser/geolocation/geolocation_infobar_delegate.h b/chrome/browser/geolocation/geolocation_infobar_delegate.h
index 6dc4b45e62f14da95e0129a29be9fc36c6c1be9c..e2eaf2c11f4246b7f11de7de4f96995825064e62 100644
--- a/chrome/browser/geolocation/geolocation_infobar_delegate.h
+++ b/chrome/browser/geolocation/geolocation_infobar_delegate.h
@@ -39,6 +39,11 @@ class GeolocationInfoBarDelegate : public ConfirmInfoBarDelegate {
   // ConfirmInfoBarDelegate:
   virtual bool Accept() OVERRIDE;
 
+  // Give the InfoBar a chance to do some work before showing
+  // to the user.
+  // Make infobar visible to the user.

[bulach, 2013/09/04 18:48:53]

not quite clear what this comment mean :)

perhaps something like:

// Override to do some work right before the infobar is
// visible to the user.
virtual BeforeShow(InfoBarService* infobar_service) {} 

then the implementation would call this before the AddInfoBar?

[acleung1, 2013/09/05 21:50:16]

Oops. Strayed comments from a delete.

I think this addresses the issue Miguel mentioned earlier about introducing too
many functions. Constructor -> show() seems simpiler

+  virtual void Show(InfoBarService* infobar_service);
+
   // Call back to the controller, to inform of the user's decision.
   void SetPermission(bool update_content_setting, bool allowed);