diff --git a/components/layout/inline.rs b/components/layout/inline.rs index 1ce18b562bbb..60edeedff4b1 100644 --- a/components/layout/inline.rs +++ b/components/layout/inline.rs @@ -962,13 +962,29 @@ impl InlineFlow { Some(ref runs) => runs[run_count - run_idx - 1], // reverse order for RTL runs None => (line.range, bidi::Level::ltr()) }; + + struct MaybeReverse { + iter: I, + reverse: bool, + } + + impl Iterator for MaybeReverse { + type Item = I::Item; + + fn next(&mut self) -> Option { + if self.reverse { + self.iter.next_back() + } else { + self.iter.next() + } + } + } + // If the bidi embedding direction is opposite the layout direction, lay out this // run in reverse order. - let reverse = level.is_ltr() != is_ltr; - let fragment_indices = if reverse { - (range.end().get() - 1..range.begin().get() - 1).step_by(-1) - } else { - (range.begin().get()..range.end().get()).step_by(1) + let fragment_indices = MaybeReverse { + iter: range.begin().get()..range.end().get(), + reverse: level.is_ltr() != is_ltr, }; for fragment_index in fragment_indices { diff --git a/components/layout/lib.rs b/components/layout/lib.rs index 0b9615509747..af108249c42a 100644 --- a/components/layout/lib.rs +++ b/components/layout/lib.rs @@ -8,7 +8,6 @@ #![feature(conservative_impl_trait)] #![feature(nonzero)] #![feature(raw)] -#![feature(step_by)] extern crate app_units; extern crate atomic_refcell; diff --git a/components/selectors/parser.rs b/components/selectors/parser.rs index c73ac10f4420..f341911a33cb 100644 --- a/components/selectors/parser.rs +++ b/components/selectors/parser.rs @@ -1572,7 +1572,6 @@ pub mod tests { use builder::HAS_PSEUDO_BIT; use cssparser::{Parser as CssParser, ToCss, serialize_identifier, ParserInput}; use parser; - use std::borrow::Cow; use std::collections::HashMap; use std::fmt; use super::*;