Fix Android strict-mode violation in GeoLocation info bar.

chrome/browser/geolocation/geolocation_infobar_delegate_android.cc

 // Copyright (c) 2012 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/geolocation/geolocation_infobar_delegate_android.h"
 
+#include "base/bind.h"
+#include "base/bind_helpers.h"
+#include "base/callback.h"
+#include "base/location.h"
 #include "base/strings/utf_string_conversions.h"
+#include "base/threading/worker_pool.h"
 #include "chrome/browser/android/google_location_settings_helper.h"
+#include "content/public/browser/browser_thread.h"
 #include "grit/generated_resources.h"
 #include "grit/locale_settings.h"
 #include "grit/theme_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 
 GeolocationInfoBarDelegateAndroid::GeolocationInfoBarDelegateAndroid(
     InfoBarService* infobar_service,
     PermissionQueueController* controller,
     const PermissionRequestID& id,
     const GURL& requesting_frame_url,
     int contents_unique_id,
     const std::string& display_languages)
     : GeolocationInfoBarDelegate(infobar_service, controller, id,
                                  requesting_frame_url, contents_unique_id,
                                  display_languages),
       google_location_settings_helper_(
-          GoogleLocationSettingsHelper::Create()) {
+          GoogleLocationSettingsHelper::Create()),
+      is_allow_label_(false) {
 }
 
 GeolocationInfoBarDelegateAndroid::~GeolocationInfoBarDelegateAndroid() {
 }
 
+void GeolocationInfoBarDelegateAndroid::Show(
+      InfoBarService* infobar_service) {
+  base::WorkerPool::PostTask(FROM_HERE,
+      base::Bind(&GeolocationInfoBarDelegateAndroid::CheckLocationSettings,
+          base::Unretained(this),

[bulach, 2013/09/12 09:18:29]

this won't work... :(
the infobar could be deleted before the worker pool executed, and then this
callback would deref a dangling pointer.. the infobar is mostly a UI object and
ideally wouldn't know anything about the underlying thread model.. also, I think
the "Accept()" would have a strict-mode violation too, since it's calling on the
google_location_settings_helper_ as well...

at the end of the day, this needs one string and one boolean :)
we should pick those up in ChromeGeolocationPermissionContextAndroid and then
plumb them through here..

[acleung1, 2013/09/17 00:08:45]

Hmm I understood the concern but I don't really follow your suggestions.

What you are suggesting that is we let ChromeGeolocationPermissionContextAndroid
do that work of CheckLocationSetting and pass these down to
GeolocationInfoBarDelegateAndroid? WOuld that introduce a dependency between
those two class that wasn't there before?

Or am I misunderstand your suggestion?

[bulach, 2013/10/16 10:47:11]

see above..

+          infobar_service), true);
+}
+
+void GeolocationInfoBarDelegateAndroid::CallParentShow(
+    InfoBarService* infobar_service) {
+  GeolocationInfoBarDelegate::Show(infobar_service);
+}
+
+void GeolocationInfoBarDelegateAndroid::CheckLocationSettings(
+    InfoBarService* infobar_service) {
+  is_allow_label_ = google_location_settings_helper_->IsAllowLabel();
+  content::BrowserThread::PostTask(content::BrowserThread::UI, FROM_HERE,
+      base::Bind(&GeolocationInfoBarDelegateAndroid::CallParentShow,
+      base::Unretained(this), infobar_service));
+}
+
 bool GeolocationInfoBarDelegateAndroid::Accept() {
   // Accept button text could be either 'Allow' or 'Google Location Settings'.
   // If 'Allow' we follow the regular flow.
   if (google_location_settings_helper_->IsGoogleAppsLocationSettingEnabled())
     return GeolocationInfoBarDelegate::Accept();
 
   // If 'Google Location Settings', we need to open the system Google Location
   // Settings activity.
   google_location_settings_helper_->ShowGoogleLocationSettings();
   SetPermission(false, false);
   return true;
 }
 
 string16 GeolocationInfoBarDelegateAndroid::GetButtonLabel(
     InfoBarButton button) const {
-  return (button == BUTTON_OK) ?
-      UTF8ToUTF16(google_location_settings_helper_->GetAcceptButtonLabel()) :
+  return (button == BUTTON_OK) ? UTF8ToUTF16(
+      google_location_settings_helper_->GetAcceptButtonLabel(is_allow_label_)) :
       l10n_util::GetStringUTF16(IDS_GEOLOCATION_DENY_BUTTON);
 }