Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright 2016 The Chromium Authors. All rights reserved. | 1 // Copyright 2016 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "core/css/cssom/CSSTokenStreamValue.h" | 5 #include "core/css/cssom/CSSTokenStreamValue.h" |
| 6 | 6 |
| 7 #include "core/css/CSSVariableReferenceValue.h" | 7 #include "core/css/CSSVariableReferenceValue.h" |
| 8 #include "core/css/parser/CSSTokenizer.h" | 8 #include "core/css/parser/CSSTokenizer.h" |
| 9 #include "wtf/text/StringBuilder.h" | 9 #include "wtf/text/StringBuilder.h" |
| 10 | 10 |
| 11 namespace blink { | 11 namespace blink { |
| 12 | 12 |
| 13 namespace { | 13 namespace { |
| 14 | 14 |
| 15 class TokenStreamValueIterationSource final : public ValueIterable<String>::Iter ationSource { | 15 class TokenStreamValueIterationSource final : public ValueIterable<StringOrCSSVa riableReferenceValue>::IterationSource { |
| 16 public: | 16 public: |
| 17 explicit TokenStreamValueIterationSource(CSSTokenStreamValue* tokenStreamVal ue) | 17 explicit TokenStreamValueIterationSource(CSSTokenStreamValue* tokenStreamVal ue) |
| 18 : m_tokenStreamValue(tokenStreamValue) | 18 : m_tokenStreamValue(tokenStreamValue) |
| 19 { | 19 { |
| 20 } | 20 } |
| 21 | 21 |
| 22 bool next(ScriptState*, String& value, ExceptionState&) override | 22 bool next(ScriptState*, StringOrCSSVariableReferenceValue& value, ExceptionS tate&) override |
| 23 { | 23 { |
| 24 if (m_index >= m_tokenStreamValue->size()) | 24 if (m_index >= m_tokenStreamValue->size()) |
| 25 return false; | 25 return false; |
| 26 value = m_tokenStreamValue->fragmentAtIndex(m_index); | 26 StringOrCSSVariableReferenceValue result; |
| 27 result.setString(m_tokenStreamValue->fragmentAtIndex(m_index)); | |
| 28 value = result; | |
|
meade_UTC10
2016/07/20 03:20:29
Instead of the above three lines, you could just w
| |
| 27 return true; | 29 return true; |
| 28 } | 30 } |
| 29 | 31 |
| 30 DEFINE_INLINE_VIRTUAL_TRACE() | 32 DEFINE_INLINE_VIRTUAL_TRACE() |
| 31 { | 33 { |
| 32 visitor->trace(m_tokenStreamValue); | 34 visitor->trace(m_tokenStreamValue); |
| 33 ValueIterable<String>::IterationSource::trace(visitor); | 35 ValueIterable<StringOrCSSVariableReferenceValue>::IterationSource::trace (visitor); |
| 34 } | 36 } |
| 35 | 37 |
| 36 private: | 38 private: |
| 37 const Member<CSSTokenStreamValue> m_tokenStreamValue; | 39 const Member<CSSTokenStreamValue> m_tokenStreamValue; |
| 38 }; | 40 }; |
| 39 | 41 |
| 40 } // namespace | 42 } // namespace |
| 41 | 43 |
| 42 ValueIterable<String>::IterationSource* CSSTokenStreamValue::startIteration(Scri ptState*, ExceptionState&) | 44 ValueIterable<StringOrCSSVariableReferenceValue>::IterationSource* CSSTokenStrea mValue::startIteration(ScriptState*, ExceptionState&) |
| 43 { | 45 { |
| 44 return new TokenStreamValueIterationSource(this); | 46 return new TokenStreamValueIterationSource(this); |
| 45 } | 47 } |
| 46 | 48 |
| 47 CSSValue* CSSTokenStreamValue::toCSSValue() const | 49 CSSValue* CSSTokenStreamValue::toCSSValue() const |
| 48 { | 50 { |
| 49 StringBuilder tokens; | 51 StringBuilder tokens; |
| 50 | 52 |
| 51 for (unsigned i = 0; i < m_fragments.size(); i++) { | 53 for (unsigned i = 0; i < m_fragments.size(); i++) { |
| 52 if (i) | 54 if (i) |
| 53 tokens.append("/**/"); | 55 tokens.append("/**/"); |
| 54 tokens.append(m_fragments[i]); | 56 tokens.append(m_fragments[i]); |
| 55 } | 57 } |
| 56 | 58 |
| 57 CSSTokenizer::Scope scope(tokens.toString()); | 59 CSSTokenizer::Scope scope(tokens.toString()); |
| 58 | 60 |
| 59 return CSSVariableReferenceValue::create(CSSVariableData::create(scope.token Range())); | 61 return CSSVariableReferenceValue::create(CSSVariableData::create(scope.token Range())); |
| 60 } | 62 } |
| 61 | 63 |
| 62 } // namespace blink | 64 } // namespace blink |
| OLD | NEW |