Factor out GamepadCommon base class from Gamepad and WebKitGamepad

Source/modules/gamepad/GamepadList.h

Index: Source/modules/gamepad/GamepadList.h
diff --git a/Source/modules/gamepad/GamepadList.h b/Source/modules/gamepad/GamepadList.h
index b19e6afc5c042d2e2ff471da76bf39ceb9734733..7f2e714d19a3186e962e69b5136dcb3bad88922c 100644
--- a/Source/modules/gamepad/GamepadList.h
+++ b/Source/modules/gamepad/GamepadList.h
@@ -36,23 +36,22 @@
 
 namespace WebCore {
 
-class GamepadList : public RefCountedWillBeGarbageCollectedFinalized<GamepadList>, public ScriptWrappable {
+class GamepadList FINAL : public RefCountedWillBeGarbageCollectedFinalized<GamepadList>, public ScriptWrappable {
 public:
-    static PassRefPtrWillBeRawPtr<GamepadList> create() { return adoptRefWillBeNoop(new GamepadList); }
-    ~GamepadList();
+    static PassRefPtrWillBeRawPtr<GamepadList> create()
+    {
+        return adoptRefWillBeNoop(new GamepadList);
+    }
+    ~GamepadList() { }
 
     void set(unsigned index, PassRefPtrWillBeRawPtr<Gamepad>);
     Gamepad* item(unsigned index);
-    unsigned length() const;
+    unsigned length() const { return blink::WebGamepads::itemsLengthCap; }

[kouhei (in TOK), 2014/03/13 09:05:11]

Is this correct? always return capacity?

[Nils Barth (inactive), 2014/03/13 09:15:14]

Yup (it's always length 4 in fact).
This is just inlined from the .cpp:
https://codereview.chromium.org/195993007/diff/80001/Source/modules/gamepad/G...

 
     void trace(Visitor*);
 
 private:
-    GamepadList()
-    {
-        ScriptWrappable::init(this);
-    }
-
+    GamepadList() { ScriptWrappable::init(this); }
     RefPtrWillBeMember<Gamepad> m_items[blink::WebGamepads::itemsLengthCap];
 };