wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Backed out changeset b7cd1bc458a3 (bug 1230172) because something from this push turned browser_video_test.js permafail on mulet CLOSED TREE

Browse Source
This commit is contained in:
Wes Kocher 2016-01-05 15:14:44 -08:00
parent 5780a15992
commit 305c250f35
6 changed files with 25 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
dom/webidl/CSS.webidl
View File

@ -20,5 +20,6 @@ interface CSS {
// http://dev.w3.org/csswg/cssom/#the-css.escape%28%29-method
partial interface CSS {
[Throws]
static DOMString escape(DOMString ident);
};

9
layout/style/CSS.cpp
View File

@ -85,9 +85,14 @@ CSS::Supports(const GlobalObject& aGlobal,
/* static */ void
CSS::Escape(const GlobalObject& aGlobal,
const nsAString& aIdent,
nsAString& aReturn)
nsAString& aReturn,
ErrorResult& aRv)
{
nsStyleUtil::AppendEscapedCSSIdent(aIdent, aReturn);
bool success = nsStyleUtil::AppendEscapedCSSIdent(aIdent, aReturn);
if (!success) {
aRv.Throw(NS_ERROR_DOM_INVALID_CHARACTER_ERR);
}
}
} // namespace dom

3
layout/style/CSS.h
View File

@ -35,7 +35,8 @@ public:
static void Escape(const GlobalObject& aGlobal,
const nsAString& aIdent,
nsAString& aReturn);
nsAString& aReturn,
ErrorResult& aRv);
};
} // namespace dom

12
layout/style/nsStyleUtil.cpp
View File

@ -79,7 +79,7 @@ void nsStyleUtil::AppendEscapedCSSString(const nsAString& aString,
aReturn.Append(quoteChar);
}
/* static */ void
/* static */ bool
nsStyleUtil::AppendEscapedCSSIdent(const nsAString& aIdent, nsAString& aReturn)
{
// The relevant parts of the CSS grammar are:
@ -98,7 +98,7 @@ nsStyleUtil::AppendEscapedCSSIdent(const nsAString& aIdent, nsAString& aReturn)
const char16_t* const end = aIdent.EndReading();
if (in == end)
return;
return true;
// A leading dash does not need to be escaped as long as it is not the
// *only* character in the identifier.
@ -106,7 +106,7 @@ nsStyleUtil::AppendEscapedCSSIdent(const nsAString& aIdent, nsAString& aReturn)
if (in + 1 == end) {
aReturn.Append(char16_t('\\'));
aReturn.Append(char16_t('-'));
return;
return true;
}
aReturn.Append(char16_t('-'));
@ -124,8 +124,9 @@ nsStyleUtil::AppendEscapedCSSIdent(const nsAString& aIdent, nsAString& aReturn)
for (; in != end; ++in) {
char16_t ch = *in;
if (ch == 0x00) {
aReturn.Append(char16_t(0xFFFD));
} else if (ch < 0x20 || (0x7F <= ch && ch < 0xA0)) {
return false;
}
if (ch < 0x20 || (0x7F <= ch && ch < 0xA0)) {
// Escape U+0000 through U+001F and U+007F through U+009F numerically.
aReturn.AppendPrintf("\\%hx ", *in);
} else {
@ -141,6 +142,7 @@ nsStyleUtil::AppendEscapedCSSIdent(const nsAString& aIdent, nsAString& aReturn)
aReturn.Append(ch);
}
}
return true;
}
// unquoted family names must be a sequence of idents

7
layout/style/nsStyleUtil.h
View File

@ -39,9 +39,10 @@ public:
// Append the identifier given by |aIdent| to |aResult|, with
// appropriate escaping so that it can be reparsed to the same
// identifier. An exception is if aIdent contains U+0000, which
// will be escaped as U+FFFD and then reparsed back to U+FFFD.
static void AppendEscapedCSSIdent(const nsAString& aIdent,
// identifier.
// Returns false if |aIdent| contains U+0000
// Returns true for all other cases
static bool AppendEscapedCSSIdent(const nsAString& aIdent,
nsAString& aResult);
static void

15
layout/style/test/test_css_escape_api.html
View File

@ -21,18 +21,11 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=955860
// Tests taken from:
// https://github.com/mathiasbynens/CSS.escape/blob/master/tests/tests.js
SimpleTest.doesThrow(() => CSS.escape('\0'), "InvalidCharacterError Character :\\0");
SimpleTest.doesThrow(() => CSS.escape('a\0'), "InvalidCharacterError Character : a\\0");
SimpleTest.doesThrow(() => CSS.escape('\0b'), "InvalidCharacterError Character : \\0b");
SimpleTest.doesThrow(() => CSS.escape('a\0b'), "InvalidCharacterError Character : a\\0b");
SimpleTest.doesThrow(() => CSS.escape(), 'undefined');
is(CSS.escape('\0'), '\uFFFD', "escaping for 0 char (1)");
is(CSS.escape('a\0'), 'a\uFFFD', "escaping for 0 char (2)");
is(CSS.escape('\0b'), '\uFFFDb', "escaping for 0 char (3)");
is(CSS.escape('a\0b'), 'a\uFFFDb', "escaping for 0 char (4)");
is(CSS.escape('\uFFFD'), '\uFFFD', "escaping for replacement char (1)");
is(CSS.escape('a\uFFFD'), 'a\uFFFD', "escaping replacement char (2)");
is(CSS.escape('\uFFFDb'), '\uFFFDb', "escaping replacement char (3)");
is(CSS.escape('a\uFFFDb'), 'a\uFFFDb', "escaping replacement char (4)");
is(CSS.escape(true), 'true', "escapingFailed Character : true(bool)");
is(CSS.escape(false), 'false', "escapingFailed Character : false(bool)");
is(CSS.escape(null), 'null', "escapingFailed Character : null");