Supresses app permissions changed dialogs in kiosk mode.

chrome/browser/usb/usb_service.cc

Index: chrome/browser/usb/usb_service.cc
diff --git a/chrome/browser/usb/usb_service.cc b/chrome/browser/usb/usb_service.cc
index fcad23d94fc78ee555ac23a2cacbb330f056096c..ec75d512c7302a4c3f5f51b28fb85119e6e14e5f 100644
--- a/chrome/browser/usb/usb_service.cc
+++ b/chrome/browser/usb/usb_service.cc
@@ -14,6 +14,7 @@
 #include "third_party/libusb/src/libusb/libusb.h"
 
 #if defined(OS_CHROMEOS)
+#include "base/chromeos/chromeos_version.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/dbus/permission_broker_client.h"
 #endif  // defined(OS_CHROMEOS)
@@ -74,22 +75,26 @@ void UsbService::FindDevices(const uint16 vendor_id,
                              const base::Callback<void()>& callback) {
   DCHECK(event_handler_) << "FindDevices called after event handler stopped.";
 #if defined(OS_CHROMEOS)
-  chromeos::PermissionBrokerClient* client =
-      chromeos::DBusThreadManager::Get()->GetPermissionBrokerClient();
-  DCHECK(client) << "Could not get permission broker client.";
-  if (!client) {
-    callback.Run();
-    return;
-  }
+  if (base::chromeos::IsRunningOnChromeOS()) {

[bryeung, 2013/05/27 12:40:54]

This looks very wrong, so I must not understand.  When can this be false given
the #if above?  Does kiosk mode trigger defined(OS_CHROMEOS) but not
IsRunningOnChromeOS?

[zel, 2013/05/27 18:48:12]

base::chromeos::IsRunningOnChromeOS() returns false when you run
GYP_DEFINES=chromeos=1 build on your LInux workstation.  It return true on the
actual CrOS hardware. So, this change is needed to support testing of
ChromeOS-flavored builds on system without PermissonBroker.

[bryeung, 2013/05/27 18:50:26]

Got it: thanks for the explanation.  A brief comment here might help prevent
confusion in the future.

[zel, 2013/05/28 16:58:27]

Done.

+    chromeos::PermissionBrokerClient* client =
+        chromeos::DBusThreadManager::Get()->GetPermissionBrokerClient();
+    DCHECK(client) << "Could not get permission broker client.";
+    if (!client) {
+      callback.Run();
+      return;
+    }
 
-  client->RequestUsbAccess(vendor_id,
-                           product_id,
-                           base::Bind(&UsbService::FindDevicesImpl,
-                                      base::Unretained(this),
-                                      vendor_id,
-                                      product_id,
-                                      devices,
-                                      callback));
+    client->RequestUsbAccess(vendor_id,
+                             product_id,
+                             base::Bind(&UsbService::FindDevicesImpl,
+                                        base::Unretained(this),
+                                        vendor_id,
+                                        product_id,
+                                        devices,
+                                        callback));
+  } else {
+    FindDevicesImpl(vendor_id, product_id, devices, callback, true);
+  }
 #else
   FindDevicesImpl(vendor_id, product_id, devices, callback, true);
 #endif  // defined(OS_CHROMEOS)