Delete WebGamepads::length.

components/exo/gamepad.cc

 // Copyright 2016 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 "components/exo/gamepad.h"
 
 #include <cmath>
 
 #include "base/bind.h"
 #include "base/location.h"
@@ skipping 105 matching lines @@
     has_poll_scheduled_ = false;
     if (!is_enabled_)
       return;
 
     DCHECK(fetcher_);
 
     blink::WebGamepads new_state = state_;
     fetcher_->GetGamepadData(
         false /* No hardware changed notification from the system */);
 
-    new_state.length = 0;
     device::PadState& pad_state = pad_states_.get()[0];
 
     // After querying the gamepad clear the state if it did not have it's active
     // state updated but is still listed as being associated with a specific
     // source. This indicates the gamepad is disconnected.
     if (!pad_state.active_state &&
         pad_state.source != device::GAMEPAD_SOURCE_NONE) {
       ClearPadState(pad_state);
     }
 
     MapAndSanitizeGamepadData(&pad_state, &new_state.items[0],
                               false /* Don't sanitize gamepad data */);
 
-    // If the gamepad was active then increment the length of the WebGamepads
-    // struct to indicate it's valid, then set the pad state to inactive. If the
-    // gamepad is still actively reporting the next call to GetGamepadData will
-    // set the active state to active again.
-    if (pad_state.active_state) {
-      new_state.length++;
+    // If the gamepad is still actively reporting the next call to
+    // GetGamepadData will set the active state to active again.
+    if (pad_state.active_state)
       pad_state.active_state = device::GAMEPAD_INACTIVE;
-    }
 
-    if (std::max(new_state.length, state_.length) > 0) {
-      if (new_state.items[0].connected != state_.items[0].connected ||
-          new_state.items[0].timestamp > state_.items[0].timestamp) {
-        origin_task_runner_->PostTask(
-            FROM_HERE,
-            base::Bind(process_gamepad_changes_, new_state.items[0]));
-      }
+    if (new_state.items[0].connected != state_.items[0].connected ||
+        new_state.items[0].timestamp > state_.items[0].timestamp) {
+      origin_task_runner_->PostTask(
+          FROM_HERE, base::Bind(process_gamepad_changes_, new_state.items[0]));
     }
 
     state_ = new_state;
     SchedulePollOnPollingThread();
   }
 
   // Callback to ProcessGamepadChanges using weak reference to Gamepad.
   ProcessGamepadChangesCallback process_gamepad_changes_;
 
   // Callback method to create a gamepad data fetcher.
@@ skipping 113 matching lines @@
       delegate_->OnButton(button_id, new_button.pressed, new_button.value);
     }
   }
   if (send_frame)
     delegate_->OnFrame();
 
   pad_state_ = new_pad;
 }
 
 }  // namespace exo