Fix use of arraysize in GamepadProvider

content/browser/gamepad/gamepad_provider.cc

Index: content/browser/gamepad/gamepad_provider.cc
diff --git a/content/browser/gamepad/gamepad_provider.cc b/content/browser/gamepad/gamepad_provider.cc
index f3493b4e7ceedaa6d8f2ec2aaef996a2bc7e1e93..850233c521838d195f97757adedfc6f8e513d50f 100644
--- a/content/browser/gamepad/gamepad_provider.cc
+++ b/content/browser/gamepad/gamepad_provider.cc
@@ -171,34 +171,35 @@ bool GamepadProvider::PadState::Match(const WebGamepad& pad) const {
   return connected_ == pad.connected &&
          axes_length_ == pad.axesLength &&
          buttons_length_ == pad.buttonsLength &&
-         memcmp(id_, pad.id, arraysize(id_)) == 0 &&
-         memcmp(mapping_, pad.mapping, arraysize(mapping_)) == 0;
+         memcmp(id_, pad.id, arraysize(id_) * sizeof(id_[0])) == 0 &&

[inferno, 2015/03/20 15:13:26]

why not just sizeof(id_) ? same at all other places ?

[kenrb, 2015/03/20 15:53:02]

I kind of like arraysize because it gives a compile error if you use it
incorrectly, but now that you point it out, it is much cleaner just to use
sizeof correctly.

+         memcmp(mapping_, pad.mapping,
+            arraysize(mapping_) * sizeof(mapping_[0])) == 0;
 }
 
 void GamepadProvider::PadState::SetPad(const WebGamepad& pad) {
   connected_ = pad.connected;
   axes_length_ = pad.axesLength;
   buttons_length_ = pad.buttonsLength;
-  memcpy(id_, pad.id, arraysize(id_));
-  memcpy(mapping_, pad.mapping, arraysize(mapping_));
+  memcpy(id_, pad.id, arraysize(id_) * sizeof(id_[0]));
+  memcpy(mapping_, pad.mapping, arraysize(mapping_) * sizeof(mapping_[0]));
 }
 
 void GamepadProvider::PadState::SetDisconnected() {
   connected_ = false;
   axes_length_ = 0;
   buttons_length_ = 0;
-  memset(id_, 0, arraysize(id_));
-  memset(mapping_, 0, arraysize(mapping_));
+  memset(id_, 0, arraysize(id_) * sizeof(id_[0]));
+  memset(mapping_, 0, arraysize(mapping_) * sizeof(mapping_[0]));
 }
 
 void GamepadProvider::PadState::AsWebGamepad(WebGamepad* pad) {
   pad->connected = connected_;
   pad->axesLength = axes_length_;
   pad->buttonsLength = buttons_length_;
-  memcpy(pad->id, id_, arraysize(id_));
-  memcpy(pad->mapping, mapping_, arraysize(mapping_));
-  memset(pad->axes, 0, arraysize(pad->axes));
-  memset(pad->buttons, 0, arraysize(pad->buttons));
+  memcpy(pad->id, id_, arraysize(id_) * sizeof(id_[0]));
+  memcpy(pad->mapping, mapping_, arraysize(mapping_) * sizeof(mapping_[0]));
+  memset(pad->axes, 0, arraysize(pad->axes) * sizeof(pad->axes[0]));
+  memset(pad->buttons, 0, arraysize(pad->buttons) * sizeof(pad->buttons[0]));
 }
 
 void GamepadProvider::DoPoll() {