Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-05 05:30:08 +01:00
Code Issues Projects Releases Packages Wiki Activity

style: Always inline GeckoElement::has_class.

Browse source 
It's easier, and probably faster, than trying to inline the function call to
Gecko.

This symbol is in libxul.so as of right now.
This commit is contained in:
Emilio Cobos Álvarez 2018-01-16 21:40:17 +01:00
parent 1e72a12265
commit e55bb3e76d
No known key found for this signature in database
GPG key ID: 056B727BB9C1027C
2 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
components/style/gecko/snapshot_helpers.rs
View file

@ -34,7 +34,7 @@ pub fn has_class<T>(
let classes = slice::from_raw_parts(list, n as usize); let classes = slice::from_raw_parts(list, n as usize);
match case_sensitivity { match case_sensitivity {
CaseSensitivity::CaseSensitive => { CaseSensitivity::CaseSensitive => {
classes.iter().any(|ptr| name == WeakAtom::new(*ptr)) classes.iter().any(|ptr| &**name == WeakAtom::new(*ptr))
} }
CaseSensitivity::AsciiCaseInsensitive => { CaseSensitivity::AsciiCaseInsensitive => {
classes.iter().any(|ptr| name.eq_ignore_ascii_case(WeakAtom::new(*ptr))) classes.iter().any(|ptr| name.eq_ignore_ascii_case(WeakAtom::new(*ptr)))

2
components/style/gecko/wrapper.rs
View file

@ -2191,7 +2191,7 @@ impl<'le> ::selectors::Element for GeckoElement<'le> {
} }
} }
#[inline] #[inline(always)]
fn has_class(&self, name: &Atom, case_sensitivity: CaseSensitivity) -> bool { fn has_class(&self, name: &Atom, case_sensitivity: CaseSensitivity) -> bool {
if !self.may_have_class() { if !self.may_have_class() {
return false; return false;