Creating Ui for Battery State Option in Chrome Os Emulator

chrome/browser/ui/webui/chromeos/emulator/device_emulator_message_handler.cc

 // Copyright 2015 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/ui/webui/chromeos/emulator/device_emulator_message_handler.h"
 
 #include <stdlib.h>
 
 #include "base/bind.h"
 #include "base/values.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/dbus/fake_power_manager_client.h"
 #include "content/public/browser/web_ui.h"
 
 namespace {
 
 // Define the name of the callback functions that will be used by JavaScript.
 const char kBluetoothDiscoverFunction[] = "requestBluetoothDiscover";
 const char kBluetoothPairFunction[] = "requestBluetoothPair";
 const char kRequestPowerInfo[] = "requestPowerInfo";
 
 // Define update functions that will update the power properties to the
 // variables defined in the web UI.
 const char kUpdateBatteryPercent[] = "updateBatteryPercent";
+const char kUpdateBatteryState[] = "updateBatteryState";
 const char kUpdateExternalPower[] = "updateExternalPower";
 const char kUpdateTimeToEmpty[] = "updateTimeToEmpty";
 const char kUpdateTimeToFull[] = "updateTimeToFull";
 
 // Define callback functions that will update the JavaScript variable
 // and the web UI.
 const char kUpdatePowerPropertiesJSCallback[] =
     "device_emulator.batterySettings.updatePowerProperties";
 
 }  // namespace
@@ skipping 36 matching lines @@
   power_manager::PowerSupplyProperties props =
       fake_power_manager_client_->props();
 
   int new_percent;
   if (args->GetInteger(0, &new_percent))
     props.set_battery_percent(new_percent);
 
   fake_power_manager_client_->UpdatePowerProperties(props);
 }
 
+void DeviceEmulatorMessageHandler::UpdateBatteryState(
+    const base::ListValue* args) {
+  power_manager::PowerSupplyProperties props =
+      fake_power_manager_client_->props();
+
+  int battery_state;
+  if (args->GetInteger(0, &battery_state))

[oshima, 2015/07/29 21:25:12]

nit: you need {} in this case

[mozartalouis, 2015/07/29 21:55:47]

Done.

+    props.set_battery_state(
+        static_cast<power_manager::PowerSupplyProperties_BatteryState>(
+            battery_state));
+
+  fake_power_manager_client_->UpdatePowerProperties(props);
+}
+
 void DeviceEmulatorMessageHandler::UpdateExternalPower(
     const base::ListValue* args) {
-  int power_source;
-  args->GetInteger(0, &power_source);
-
   power_manager::PowerSupplyProperties props =
       fake_power_manager_client_->props();
+
+  int power_source;
+  if (args->GetInteger(0, &power_source))
   props.set_external_power(

[oshima, 2015/07/29 21:25:12]

nit: indent and {}

[mozartalouis, 2015/07/29 21:55:47]

Done.

       static_cast<power_manager::PowerSupplyProperties_ExternalPower>(
           power_source));
+
   fake_power_manager_client_->UpdatePowerProperties(props);
 }
 
 void DeviceEmulatorMessageHandler::UpdateTimeToEmpty(
     const base::ListValue* args) {
   power_manager::PowerSupplyProperties props =
       fake_power_manager_client_->props();
 
   int new_time;
   if (args->GetInteger(0, &new_time))
     props.set_battery_time_to_empty_sec(new_time);
 
   fake_power_manager_client_->UpdatePowerProperties(props);
 }
 
 void DeviceEmulatorMessageHandler::UpdateTimeToFull(
     const base::ListValue* args) {
   power_manager::PowerSupplyProperties props =
       fake_power_manager_client_->props();
+
   int new_time;
   if (args->GetInteger(0, &new_time))
     props.set_battery_time_to_full_sec(new_time);
+
   fake_power_manager_client_->UpdatePowerProperties(props);
 }
 
 void DeviceEmulatorMessageHandler::PowerChanged(
     const power_manager::PowerSupplyProperties& proto) {
-  web_ui()->CallJavascriptFunction(
-      kUpdatePowerPropertiesJSCallback,
-      base::FundamentalValue(proto.battery_percent()),
-      base::FundamentalValue(proto.external_power()),
-      base::FundamentalValue(
-          static_cast<int>(proto.battery_time_to_empty_sec())),
-      base::FundamentalValue(
-          static_cast<int>(proto.battery_time_to_full_sec())));
+  // Create a list that holds all the values that the ui will need
+  // when this function is called.

[oshima, 2015/07/29 21:25:13]

This comment isn't much useful. You can omit it.

[mozartalouis, 2015/07/29 21:55:47]

Done.

+  scoped_ptr<base::ListValue> power_properties(new base::ListValue);

[oshima, 2015/07/29 21:25:13]

you can just create an object on stack like

base::ListValue power_properties;

[mozartalouis, 2015/07/29 21:55:47]

Done.

+
+  power_properties->AppendInteger(static_cast<int>(proto.battery_percent()));
+  power_properties->AppendInteger(static_cast<int>(proto.battery_state()));
+  power_properties->AppendInteger(static_cast<int>(proto.external_power()));
+  power_properties->AppendInteger(
+      static_cast<int>(proto.battery_time_to_empty_sec()));
+  power_properties->AppendInteger(
+      static_cast<int>(proto.battery_time_to_full_sec()));
+
+  web_ui()->CallJavascriptFunction(kUpdatePowerPropertiesJSCallback,
+                                   *power_properties);

[oshima, 2015/07/29 21:25:13]

I thought you wanted to use

CallJavascriptFunction(
   const std::string& function_name,
   const std::vector<const base::Value*>& args)

?

If you want to just pass as one argument (which is probably better IMO), using
dictionary with the same key names with proto buffers will make the code more
readable, instead of using list, and access using index.

[mozartalouis, 2015/07/29 21:55:47]

Done.

 }
 
 void DeviceEmulatorMessageHandler::RegisterMessages() {
   web_ui()->RegisterMessageCallback(
       kRequestPowerInfo,
       base::Bind(&DeviceEmulatorMessageHandler::RequestPowerInfo,
                  base::Unretained(this)));
   web_ui()->RegisterMessageCallback(
       kUpdateBatteryPercent,
       base::Bind(&DeviceEmulatorMessageHandler::UpdateBatteryPercent,
                  base::Unretained(this)));
   web_ui()->RegisterMessageCallback(
+      kUpdateBatteryState,
+      base::Bind(&DeviceEmulatorMessageHandler::UpdateBatteryState,
+                 base::Unretained(this)));
+  web_ui()->RegisterMessageCallback(
       kUpdateExternalPower,
       base::Bind(&DeviceEmulatorMessageHandler::UpdateExternalPower,
                  base::Unretained(this)));
   web_ui()->RegisterMessageCallback(
       kUpdateTimeToEmpty,
       base::Bind(&DeviceEmulatorMessageHandler::UpdateTimeToEmpty,
                  base::Unretained(this)));
   web_ui()->RegisterMessageCallback(
       kUpdateTimeToFull,
       base::Bind(&DeviceEmulatorMessageHandler::UpdateTimeToFull,
                  base::Unretained(this)));
   web_ui()->RegisterMessageCallback(
       kBluetoothDiscoverFunction,
       base::Bind(&DeviceEmulatorMessageHandler::HandleRequestBluetoothDiscover,
                  base::Unretained(this)));
   web_ui()->RegisterMessageCallback(
       kBluetoothPairFunction,
       base::Bind(&DeviceEmulatorMessageHandler::HandleRequestBluetoothPair,
                  base::Unretained(this)));
 }