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

Use FnvHashmap instead of HashMap for AnimationValueMap.

Browse source
This commit is contained in:
Hiroyuki Ikezoe 2017-06-21 08:36:47 +09:00
parent b50cf33efd
commit 6c69771ea5
2 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

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

@ -616,7 +616,7 @@ fn get_animation_rule(element: &GeckoElement,
-> Option<Arc<Locked<PropertyDeclarationBlock>>> { -> Option<Arc<Locked<PropertyDeclarationBlock>>> {
use gecko_bindings::sugar::ownership::HasSimpleFFI; use gecko_bindings::sugar::ownership::HasSimpleFFI;
// Also, we should try to reuse the PDB, to avoid creating extra rule nodes. // Also, we should try to reuse the PDB, to avoid creating extra rule nodes.
let mut animation_values = AnimationValueMap::new(); let mut animation_values = AnimationValueMap::default();
if unsafe { Gecko_GetAnimationRule(element.0, if unsafe { Gecko_GetAnimationRule(element.0,
cascade_level, cascade_level,
AnimationValueMap::as_ffi_mut(&mut animation_values)) } { AnimationValueMap::as_ffi_mut(&mut animation_values)) } {

4
components/style/properties/helpers/animated_properties.mako.rs
View file

@ -30,7 +30,7 @@ use properties::longhands::visibility::computed_value::T as Visibility;
use selectors::parser::SelectorParseError; use selectors::parser::SelectorParseError;
use smallvec::SmallVec; use smallvec::SmallVec;
use std::cmp; use std::cmp;
#[cfg(feature = "gecko")] use std::collections::HashMap; #[cfg(feature = "gecko")] use fnv::FnvHashMap;
use style_traits::ParseError; use style_traits::ParseError;
use super::ComputedValues; use super::ComputedValues;
use values::{Auto, CSSFloat, CustomIdent, Either}; use values::{Auto, CSSFloat, CustomIdent, Either};
@ -440,7 +440,7 @@ impl AnimatedProperty {
/// This HashMap stores the values that are the last AnimationValue to be /// This HashMap stores the values that are the last AnimationValue to be
/// composed for each TransitionProperty. /// composed for each TransitionProperty.
#[cfg(feature = "gecko")] #[cfg(feature = "gecko")]
pub type AnimationValueMap = HashMap<AnimatableLonghand, AnimationValue>; pub type AnimationValueMap = FnvHashMap<AnimatableLonghand, AnimationValue>;
#[cfg(feature = "gecko")] #[cfg(feature = "gecko")]
unsafe impl HasFFI for AnimationValueMap { unsafe impl HasFFI for AnimationValueMap {
type FFIType = RawServoAnimationValueMap; type FFIType = RawServoAnimationValueMap;