Stop clamping tabIndex to short range

third_party/WebKit/Source/core/dom/ElementRareData.h

 /*
  * Copyright (C) 2008, 2009, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2008 David Smith <catfish.man@gmail.com>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 42 matching lines @@
  public:
   static ElementRareData* create(LayoutObject* layoutObject) {
     return new ElementRareData(layoutObject);
   }
 
   ~ElementRareData();
 
   void setPseudoElement(PseudoId, PseudoElement*);
   PseudoElement* pseudoElement(PseudoId) const;
 
-  short tabIndex() const { return m_tabindex; }
-
-  void setTabIndexExplicitly(short index) {
-    m_tabindex = index;
+  void setTabIndexExplicitly(int index) {
     setElementFlag(TabIndexWasSetExplicitly, true);

[foolip, 2016/11/03 14:17:42]

Is this flag actually needed? Can't hasAttribute(tabindexAttr) be used instead?

[rwlbuis, 2016/11/03 15:33:09]

I still think we need the flag. If we do setAttribute("tabIndex",
crazy_big_number), the attribute will be set.
If we change Element::supportsFocus to test for hasAttribute instead of the
flag, in this
case the element with out-of-range tabIndex would still be focusable, which I
think we do not want to allow.

[foolip, 2016/11/03 16:06:57]

<button tabindex=18446744073709552001> seems to already be focusable, or am I
testing it wrong? What do other implementations do?

   }
 
   void clearTabIndexExplicitly() {
-    m_tabindex = 0;
     clearElementFlag(TabIndexWasSetExplicitly);
   }
 
   CSSStyleDeclaration& ensureInlineCSSStyleDeclaration(Element* ownerElement);
   InlineStylePropertyMap& ensureInlineStylePropertyMap(Element* ownerElement);
 
   InlineStylePropertyMap* inlineStylePropertyMap() {
     return m_cssomMapWrapper.get();
   }
 
@@ skipping 103 matching lines @@
   }
   ResizeObserverDataMap& ensureResizeObserverData();
 
   DECLARE_TRACE_AFTER_DISPATCH();
   DECLARE_TRACE_WRAPPERS_AFTER_DISPATCH();
 
  private:
   CompositorProxiedPropertySet& ensureCompositorProxiedPropertySet();
   void clearCompositorProxiedPropertySet() { m_proxiedProperties = nullptr; }
 
-  short m_tabindex;
-
   LayoutSize m_minimumSizeForResizing;
   ScrollOffset m_savedLayerScrollOffset;
 
   Member<DatasetDOMStringMap> m_dataset;
   Member<ElementShadow> m_shadow;
   Member<ClassList> m_classList;
   Member<NamedNodeMap> m_attributeMap;
   Member<AttrNodeList> m_attrNodeList;
   Member<InlineCSSStyleDeclaration> m_cssomWrapper;
   Member<InlineStylePropertyMap> m_cssomMapWrapper;
@@ skipping 12 matching lines @@
 
   explicit ElementRareData(LayoutObject*);
 };
 
 inline LayoutSize defaultMinimumSizeForResizing() {
   return LayoutSize(LayoutUnit::max(), LayoutUnit::max());
 }
 
 inline ElementRareData::ElementRareData(LayoutObject* layoutObject)
     : NodeRareData(layoutObject),
-      m_tabindex(0),
       m_minimumSizeForResizing(defaultMinimumSizeForResizing()),
       m_classList(nullptr) {
   m_isElementRareData = true;
 }
 
 inline ElementRareData::~ElementRareData() {
   DCHECK(!m_pseudoElementData);
 }
 
 inline bool ElementRareData::hasPseudoElements() const {
@@ skipping 35 matching lines @@
 inline CompositorProxiedPropertySet&
 ElementRareData::ensureCompositorProxiedPropertySet() {
   if (!m_proxiedProperties)
     m_proxiedProperties = CompositorProxiedPropertySet::create();
   return *m_proxiedProperties;
 }
 
 }  // namespace blink
 
 #endif  // ElementRareData_h