Backed out changeset fec394734f83 (bug 17624) for build bustage a=bac…

…kout CLOSED TREE Backs out #17624
servo · Jul 6, 2017 · 32269fa · 32269fa
1 parent 585468d
commit 32269fa
Show file tree

Hide file tree

Showing 30 changed files with 1,431 additions and 3,019 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/components/canvas/Cargo.toml b/components/canvas/Cargo.toml
@@ -12,7 +12,7 @@ path = "lib.rs"
 [dependencies]
 azure = {git = "https://github.com/servo/rust-azure"}
 canvas_traits = {path = "../canvas_traits"}
-cssparser = "0.17.0"
+cssparser = "0.16.1"
 euclid = "0.15"
 gleam = "0.4"
 ipc-channel = "0.8"

diff --git a/components/canvas_traits/Cargo.toml b/components/canvas_traits/Cargo.toml
@@ -10,7 +10,7 @@ name = "canvas_traits"
 path = "lib.rs"
 
 [dependencies]
-cssparser = "0.17.0"
+cssparser = "0.16.1"
 euclid = "0.15"
 heapsize = "0.4"
 heapsize_derive = "0.1"

diff --git a/components/script/Cargo.toml b/components/script/Cargo.toml
@@ -35,7 +35,7 @@ byteorder = "1.0"
 canvas_traits = {path = "../canvas_traits"}
 caseless = "0.1.0"
 cookie = "0.6"
-cssparser = "0.17.0"
+cssparser = "0.16.1"
 deny_public_fields = {path = "../deny_public_fields"}
 devtools_traits = {path = "../devtools_traits"}
 dom_struct = {path = "../dom_struct"}

diff --git a/components/script_layout_interface/Cargo.toml b/components/script_layout_interface/Cargo.toml
@@ -13,7 +13,7 @@ path = "lib.rs"
 app_units = "0.5"
 atomic_refcell = "0.1"
 canvas_traits = {path = "../canvas_traits"}
-cssparser = "0.17.0"
+cssparser = "0.16.1"
 euclid = "0.15"
 gfx_traits = {path = "../gfx_traits"}
 heapsize = "0.4"

diff --git a/components/script_layout_interface/wrapper_traits.rs b/components/script_layout_interface/wrapper_traits.rs
@@ -415,8 +415,7 @@ pub trait ThreadSafeLayoutElement: Clone + Copy + Sized + Debug +
                             &style_pseudo,
                             Some(data.styles.primary()),
                             CascadeFlags::empty(),
-                            &ServoMetricsProvider,
-                            context.error_reporter)
+                            &ServoMetricsProvider)
                             .clone()
                     }
                     PseudoElementCascadeType::Lazy => {
@@ -428,8 +427,7 @@ pub trait ThreadSafeLayoutElement: Clone + Copy + Sized + Debug +
                                    RuleInclusion::All,
                                    data.styles.primary(),
                                    /* is_probe = */ false,
-                                   &ServoMetricsProvider,
-                                   context.error_reporter)
+                                   &ServoMetricsProvider)
                                .unwrap()
                                .clone()
                     }

diff --git a/components/selectors/Cargo.toml b/components/selectors/Cargo.toml
@@ -25,7 +25,7 @@ unstable = []
 [dependencies]
 bitflags = "0.7"
 matches = "0.1"
-cssparser = "0.17.0"
+cssparser = "0.16.1"
 log = "0.3"
 fnv = "1.0"
 phf = "0.7.18"

diff --git a/components/selectors/parser.rs b/components/selectors/parser.rs
@@ -59,7 +59,7 @@ pub enum SelectorParseError<'i, T> {
     PseudoElementExpectedIdent,
     UnsupportedPseudoClass,
     UnexpectedIdent(CompactCowStr<'i>),
-    ExpectedNamespace(CompactCowStr<'i>),
+    ExpectedNamespace,
     Custom(T),
 }
 
@@ -1105,10 +1105,9 @@ fn parse_qualified_name<'i, 't, P, E, Impl>
             let position = input.position();
             match input.next_including_whitespace() {
                 Ok(Token::Delim('|')) => {
-                    let prefix = from_cow_str(value.clone().into());
+                    let prefix = from_cow_str(value.into());
                     let result = parser.namespace_for_prefix(&prefix);
-                    let url = result.ok_or(ParseError::Custom(
-                        SelectorParseError::ExpectedNamespace(value.into())))?;
+                    let url = result.ok_or(ParseError::Custom(SelectorParseError::ExpectedNamespace))?;
                     explicit_namespace(input, QNamePrefix::ExplicitNamespace(prefix, url))
                 },
                 _ => {

diff --git a/components/style/Cargo.toml b/components/style/Cargo.toml
@@ -38,7 +38,7 @@ bitflags = "0.7"
 bit-vec = "0.4.3"
 byteorder = "1.0"
 cfg-if = "0.1.0"
-cssparser = "0.17.0"
+cssparser = "0.16.1"
 encoding = {version = "0.2", optional = true}
 euclid = "0.15"
 fnv = "1.0"

diff --git a/components/style/custom_properties.rs b/components/style/custom_properties.rs
@@ -280,10 +280,10 @@ fn parse_declaration_value_block<'i, 't>
                 }
                 token.serialization_type()
             }
-            Token::BadUrl(u) =>
-                return Err(StyleParseError::BadUrlInDeclarationValueBlock(u).into()),
-            Token::BadString(s) =>
-                return Err(StyleParseError::BadStringInDeclarationValueBlock(s).into()),
+            Token::BadUrl =>
+                return Err(StyleParseError::BadUrlInDeclarationValueBlock.into()),
+            Token::BadString =>
+                return Err(StyleParseError::BadStringInDeclarationValueBlock.into()),
             Token::CloseParenthesis =>
                 return Err(StyleParseError::UnbalancedCloseParenthesisInDeclarationValueBlock.into()),
             Token::CloseSquareBracket =>

diff --git a/components/style/error_reporting.rs b/components/style/error_reporting.rs
@@ -8,9 +8,6 @@
 
 use cssparser::{Parser, SourcePosition, BasicParseError, Token};
 use cssparser::ParseError as CssParseError;
-#[cfg(feature = "gecko")] use gecko::error_reporter::ErrorReporter;
-#[cfg(feature = "gecko")] use gecko_bindings::structs::Loader;
-#[cfg(feature = "gecko")] use gecko_bindings::structs::ServoStyleSheet;
 use log;
 use style_traits::ParseError;
 use stylesheets::UrlExtraData;
@@ -79,8 +76,8 @@ impl<'a> ContextualParseError<'a> {
                 Token::ParenthesisBlock => format!("parenthesis ("),
                 Token::SquareBracketBlock => format!("square bracket ["),
                 Token::CurlyBracketBlock => format!("curly bracket {{"),
-                Token::BadUrl(ref _u) => format!("bad url parse error"),
-                Token::BadString(ref _s) => format!("bad string parse error"),
+                Token::BadUrl => format!("bad url parse error"),
+                Token::BadString => format!("bad string parse error"),
                 Token::CloseParenthesis => format!("unmatched close parenthesis"),
                 Token::CloseSquareBracket => format!("unmatched close square bracket"),
                 Token::CloseCurlyBracket => format!("unmatched close curly bracket"),
@@ -91,11 +88,11 @@ impl<'a> ContextualParseError<'a> {
             match *err {
                 CssParseError::Basic(BasicParseError::UnexpectedToken(ref t)) =>
                     format!("found unexpected {}", token_to_str(t)),
+                CssParseError::Basic(BasicParseError::ExpectedToken(ref t)) =>
+                    format!("expected {}", token_to_str(t)),
                 CssParseError::Basic(BasicParseError::EndOfInput) =>
                     format!("unexpected end of input"),
-                CssParseError::Basic(BasicParseError::AtRuleInvalid(ref i)) =>
-                    format!("@ rule invalid: {}", i),
-                CssParseError::Basic(BasicParseError::AtRuleBodyInvalid) =>
+                CssParseError::Basic(BasicParseError::AtRuleInvalid) =>
                     format!("@ rule invalid"),
                 CssParseError::Basic(BasicParseError::QualifiedRuleInvalid) =>
                     format!("qualified rule invalid"),
@@ -193,15 +190,6 @@ impl ParseErrorReporter for NullReporter {
 }
 
 /// Create an instance of the default error reporter.
-#[cfg(feature = "servo")]
 pub fn create_error_reporter() -> RustLogReporter {
     RustLogReporter
 }
-
- /// Create an instance of the default error reporter for Stylo.
- #[cfg(feature = "gecko")]
-pub fn create_error_reporter(sheet: *mut ServoStyleSheet,
-                             loader: *mut Loader)
-                             -> ErrorReporter {
-    ErrorReporter::new(loader, sheet)
-}