Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(223)

Unified Diff: third_party/WebKit/Source/core/css/parser/CSSParserFastPaths.cpp

Issue 1858753003: Remove RawPtr from core/css (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/css/parser/CSSParserFastPaths.cpp
diff --git a/third_party/WebKit/Source/core/css/parser/CSSParserFastPaths.cpp b/third_party/WebKit/Source/core/css/parser/CSSParserFastPaths.cpp
index 708f4e200287929bb137b11c786adcdba9785066..b4ad35cdfb3f3e50852bf23e397ce23c721bb749 100644
--- a/third_party/WebKit/Source/core/css/parser/CSSParserFastPaths.cpp
+++ b/third_party/WebKit/Source/core/css/parser/CSSParserFastPaths.cpp
@@ -86,7 +86,7 @@ static inline bool parseSimpleLength(const CharacterType* characters, unsigned l
return ok && CSSPropertyParser::isValidNumericValue(number);
}
-static RawPtr<CSSValue> parseSimpleLengthValue(CSSPropertyID propertyId, const String& string, CSSParserMode cssParserMode)
+static CSSValue* parseSimpleLengthValue(CSSPropertyID propertyId, const String& string, CSSParserMode cssParserMode)
{
ASSERT(!string.isEmpty());
bool acceptsNegativeNumbers = false;
@@ -437,7 +437,7 @@ static bool fastParseColorInternal(RGBA32& rgb, const CharacterType* characters,
return false;
}
-RawPtr<CSSValue> CSSParserFastPaths::parseColor(const String& string, CSSParserMode parserMode)
+CSSValue* CSSParserFastPaths::parseColor(const String& string, CSSParserMode parserMode)
{
ASSERT(!string.isEmpty());
CSSParserString cssString;
@@ -830,7 +830,7 @@ bool CSSParserFastPaths::isKeywordPropertyID(CSSPropertyID propertyId)
}
}
-static RawPtr<CSSValue> parseKeywordValue(CSSPropertyID propertyId, const String& string)
+static CSSValue* parseKeywordValue(CSSPropertyID propertyId, const String& string)
{
ASSERT(!string.isEmpty());
@@ -902,7 +902,7 @@ static bool parseTransformNumberArguments(CharType*& pos, CharType* end, unsigne
}
template <typename CharType>
-static RawPtr<CSSFunctionValue> parseSimpleTransformValue(CharType*& pos, CharType* end)
+static CSSFunctionValue* parseSimpleTransformValue(CharType*& pos, CharType* end)
{
static const int shortestValidTransformStringLength = 12;
@@ -942,10 +942,10 @@ static RawPtr<CSSFunctionValue> parseSimpleTransformValue(CharType*& pos, CharTy
return nullptr;
}
pos += argumentStart;
- RawPtr<CSSFunctionValue> transformValue = CSSFunctionValue::create(transformType);
- if (!parseTransformTranslateArguments(pos, end, expectedArgumentCount, transformValue.get()))
+ CSSFunctionValue* transformValue = CSSFunctionValue::create(transformType);
+ if (!parseTransformTranslateArguments(pos, end, expectedArgumentCount, transformValue))
return nullptr;
- return transformValue.release();
+ return transformValue;
}
const bool isMatrix3d = toASCIILower(pos[0]) == 'm'
@@ -960,10 +960,10 @@ static RawPtr<CSSFunctionValue> parseSimpleTransformValue(CharType*& pos, CharTy
if (isMatrix3d) {
pos += 9;
- RawPtr<CSSFunctionValue> transformValue = CSSFunctionValue::create(CSSValueMatrix3d);
- if (!parseTransformNumberArguments(pos, end, 16, transformValue.get()))
+ CSSFunctionValue* transformValue = CSSFunctionValue::create(CSSValueMatrix3d);
+ if (!parseTransformNumberArguments(pos, end, 16, transformValue))
return nullptr;
- return transformValue.release();
+ return transformValue;
}
const bool isScale3d = toASCIILower(pos[0]) == 's'
@@ -977,37 +977,37 @@ static RawPtr<CSSFunctionValue> parseSimpleTransformValue(CharType*& pos, CharTy
if (isScale3d) {
pos += 8;
- RawPtr<CSSFunctionValue> transformValue = CSSFunctionValue::create(CSSValueScale3d);
- if (!parseTransformNumberArguments(pos, end, 3, transformValue.get()))
+ CSSFunctionValue* transformValue = CSSFunctionValue::create(CSSValueScale3d);
+ if (!parseTransformNumberArguments(pos, end, 3, transformValue))
return nullptr;
- return transformValue.release();
+ return transformValue;
}
return nullptr;
}
template <typename CharType>
-static RawPtr<CSSValueList> parseSimpleTransformList(CharType*& pos, CharType* end)
+static CSSValueList* parseSimpleTransformList(CharType*& pos, CharType* end)
{
- RawPtr<CSSValueList> transformList = nullptr;
+ CSSValueList* transformList = nullptr;
while (pos < end) {
while (pos < end && isCSSSpace(*pos))
++pos;
- RawPtr<CSSFunctionValue> transformValue = parseSimpleTransformValue(pos, end);
+ CSSFunctionValue* transformValue = parseSimpleTransformValue(pos, end);
if (!transformValue)
return nullptr;
if (!transformList)
transformList = CSSValueList::createSpaceSeparated();
- transformList->append(transformValue.release());
+ transformList->append(transformValue);
if (pos < end) {
if (isCSSSpace(*pos))
return nullptr;
}
}
- return transformList.release();
+ return transformList;
}
-static RawPtr<CSSValue> parseSimpleTransform(CSSPropertyID propertyID, const String& string)
+static CSSValue* parseSimpleTransform(CSSPropertyID propertyID, const String& string)
{
ASSERT(!string.isEmpty());
@@ -1023,16 +1023,16 @@ static RawPtr<CSSValue> parseSimpleTransform(CSSPropertyID propertyID, const Str
return parseSimpleTransformList(pos, end);
}
-RawPtr<CSSValue> CSSParserFastPaths::maybeParseValue(CSSPropertyID propertyID, const String& string, CSSParserMode parserMode)
+CSSValue* CSSParserFastPaths::maybeParseValue(CSSPropertyID propertyID, const String& string, CSSParserMode parserMode)
{
- if (RawPtr<CSSValue> length = parseSimpleLengthValue(propertyID, string, parserMode))
- return length.release();
+ if (CSSValue* length = parseSimpleLengthValue(propertyID, string, parserMode))
+ return length;
if (isColorPropertyID(propertyID))
return parseColor(string, parserMode);
- if (RawPtr<CSSValue> keyword = parseKeywordValue(propertyID, string))
- return keyword.release();
- if (RawPtr<CSSValue> transform = parseSimpleTransform(propertyID, string))
- return transform.release();
+ if (CSSValue* keyword = parseKeywordValue(propertyID, string))
+ return keyword;
+ if (CSSValue* transform = parseSimpleTransform(propertyID, string))
+ return transform;
return nullptr;
}

Powered by Google App Engine
This is Rietveld 408576698