Change a bool type to an enum type in constructor argument of SpaceSplitString.

Source/core/dom/SpaceSplitString.h

 /*
  * Copyright (C) 2007, 2008, 2010, 2011, 2012 Apple Inc. All rights reserved.
  *
  * 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,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Library General Public License for more details.
  *
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
  * Boston, MA 02111-1307, USA.
  *
  */
 
 #ifndef SpaceSplitString_h
 #define SpaceSplitString_h
 
 #include "wtf/RefCounted.h"
 #include "wtf/Vector.h"
 #include "wtf/text/AtomicString.h"
 
 namespace blink {
 
+enum SplitStringCase {

[Mike West, 2015/03/26 19:33:05]

I'd prefer you to move this into SpaceSplitString rather than adding it to the
global 'blink::' namespace. Perhaps `SpaceSplitString::CaseFolding`?

+    ShouldNotFoldCase,
+    ShouldFoldCase
+};
+
 class SpaceSplitString {
 public:
     SpaceSplitString() { }
-    SpaceSplitString(const AtomicString& string, bool shouldFoldCase) { set(string, shouldFoldCase); }
+    SpaceSplitString(const AtomicString& string, SplitStringCase splitStringCase) { set(string, splitStringCase); }
 
     bool operator!=(const SpaceSplitString& other) const { return m_data != other.m_data; }
 
-    void set(const AtomicString&, bool shouldFoldCase);
+    void set(const AtomicString&, SplitStringCase);
     void clear() { m_data.clear(); }
 
     bool contains(const AtomicString& string) const { return m_data && m_data->contains(string); }
     bool containsAll(const SpaceSplitString& names) const { return !names.m_data || (m_data && m_data->containsAll(*names.m_data)); }
     void add(const AtomicString&);
     bool remove(const AtomicString&);
 
     size_t size() const { return m_data ? m_data->size() : 0; }
     bool isNull() const { return !m_data; }
     const AtomicString& operator[](size_t i) const { ASSERT_WITH_SECURITY_IMPLICATION(i < size()); return (*m_data)[i]; }
@@ skipping 45 matching lines @@
         if (m_data && !m_data->isUnique())
             m_data = Data::createUnique(*m_data);
     }
 
     RefPtr<Data> m_data;
 };
 
 } // namespace blink
 
 #endif // SpaceSplitString_h