UMA metrics added for HID detection dialog.

chrome/browser/ui/webui/chromeos/login/hid_detection_screen_handler.h

 // Copyright 2014 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.
 
 #ifndef CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_HID_DETECTION_SCREEN_HANDLER_H_
 #define CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_HID_DETECTION_SCREEN_HANDLER_H_
 
 #include <string>
 #include <vector>
 
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
+#include "base/prefs/pref_registry_simple.h"
 #include "base/values.h"
 #include "chrome/browser/chromeos/device/input_service_proxy.h"
 #include "chrome/browser/chromeos/login/screens/hid_detection_screen_actor.h"
 #include "chrome/browser/ui/webui/chromeos/login/base_screen_handler.h"
 #include "content/public/browser/web_ui.h"
 #include "device/bluetooth/bluetooth_adapter.h"
 #include "device/bluetooth/bluetooth_device.h"
 #include "device/bluetooth/bluetooth_discovery_session.h"
 
 namespace base {
@@ skipping 47 matching lines @@
                            device::BluetoothDevice* device) OVERRIDE;
   virtual void DeviceChanged(device::BluetoothAdapter* adapter,
                              device::BluetoothDevice* device) OVERRIDE;
   virtual void DeviceRemoved(device::BluetoothAdapter* adapter,
                              device::BluetoothDevice* device) OVERRIDE;
 
   // InputServiceProxy::Observer implementation.
   virtual void OnInputDeviceAdded(const InputDeviceInfo& info) OVERRIDE;
   virtual void OnInputDeviceRemoved(const std::string& id) OVERRIDE;
 
+  // Registers the preference for derelict state.
+  static void RegisterPrefs(PrefRegistrySimple* registry);
+
  private:
   void InitializeAdapter(scoped_refptr<device::BluetoothAdapter> adapter);
 
   // Sends a notification to the Web UI of the status of available Bluetooth/USB
   // pointing device.
   void SendPointingDeviceNotification();
 
   // Sends a notification to the Web UI of the status of available Bluetooth/USB
   // keyboard device.
   void SendKeyboardDeviceNotification(base::DictionaryValue* params);
@@ skipping 72 matching lines @@
   InputDeviceInfo::Type pointing_device_connect_type_;
 
   // Current keyboard device, if any.
   std::string keyboard_device_name_;
   std::string keyboard_device_id_;
   bool keyboard_is_pairing_;
   InputDeviceInfo::Type keyboard_device_connect_type_;
 
   bool switch_on_adapter_when_ready_;
 
-  bool skip_screen_if_devices_present_;
+  bool first_time_screen_show_;
 
   base::WeakPtrFactory<HIDDetectionScreenHandler> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(HIDDetectionScreenHandler);
 };
 
+namespace hid {
+
+enum ContinueScenarioType {

[ygorshenin1, 2014/05/14 09:35:36]

As you're using this enum only in HIDDetectionScreenHandler could you please
move the enum declaration to the public part of HIDDetectionScreenHandler?

[merkulova, 2014/05/14 13:11:05]

Done.

+

[ygorshenin1, 2014/05/14 09:35:36]

nit: delete blank line.

[merkulova, 2014/05/14 13:11:05]

Done.

+  // Only pointing device detected, user pressed 'Continue'.
+  POINTING_DEVICE_ONLY_DETECTED,
+
+  // Only keyboard detected, user pressed 'Continue'.
+  KEYBOARD_DEVICE_ONLY_DETECTED,
+
+  // All devices detected.
+  All_DEVICES_DETECTED,
+
+  // Must be last enum element.
+  CONTINUE_SCENARIO_TYPE_SIZE
+};
+
+}  // namespace hid
+
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_HID_DETECTION_SCREEN_HANDLER_H_