aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
Diffstat
-rw-r--r--components/script/dom/bindings/trace.rs2
-rw-r--r--components/script/dom/cssconditionrule.rs5
-rw-r--r--components/script/dom/cssgroupingrule.rs7
-rw-r--r--components/script/dom/cssrulelist.rs16
-rw-r--r--components/script/dom/cssstylesheet.rs5
-rw-r--r--components/script/dom/htmlmetaelement.rs3
-rw-r--r--components/script/stylesheet_loader.rs2
-rw-r--r--components/style/encoding_support.rs4
-rw-r--r--components/style/gecko/arc_types.rs2
-rw-r--r--components/style/stylesheets.rs52
-rw-r--r--components/style/stylist.rs2
-rw-r--r--ports/geckolib/glue.rs30
-rw-r--r--tests/unit/style/media_queries.rs2
-rw-r--r--tests/unit/style/rule_tree/bench.rs3
-rw-r--r--tests/unit/style/stylesheets.rs2
15 files changed, 85 insertions, 52 deletions
diff --git a/components/script/dom/bindings/trace.rs b/components/script/dom/bindings/trace.rs
index b0fabb5fd45..122823f6f0d 100644
--- a/components/script/dom/bindings/trace.rs
+++ b/components/script/dom/bindings/trace.rs
@@ -508,7 +508,7 @@ unsafe impl JSTraceable for RwLock<FontFaceRule> {
}
}
-unsafe impl JSTraceable for RwLock<CssRules> {
+unsafe impl JSTraceable for StyleLocked<CssRules> {
unsafe fn trace(&self, _trc: *mut JSTracer) {
// Do nothing.
}
diff --git a/components/script/dom/cssconditionrule.rs b/components/script/dom/cssconditionrule.rs
index 44d10c21dd3..8bf5dad6c93 100644
--- a/components/script/dom/cssconditionrule.rs
+++ b/components/script/dom/cssconditionrule.rs
@@ -10,9 +10,8 @@ use dom::cssmediarule::CSSMediaRule;
use dom::cssstylesheet::CSSStyleSheet;
use dom::csssupportsrule::CSSSupportsRule;
use dom_struct::dom_struct;
-use parking_lot::RwLock;
use std::sync::Arc;
-use style::shared_lock::SharedRwLock;
+use style::shared_lock::{SharedRwLock, Locked};
use style::stylesheets::CssRules as StyleCssRules;
#[dom_struct]
@@ -22,7 +21,7 @@ pub struct CSSConditionRule {
impl CSSConditionRule {
pub fn new_inherited(parent_stylesheet: &CSSStyleSheet,
- rules: Arc<RwLock<StyleCssRules>>) -> CSSConditionRule {
+ rules: Arc<Locked<StyleCssRules>>) -> CSSConditionRule {
CSSConditionRule {
cssgroupingrule: CSSGroupingRule::new_inherited(parent_stylesheet, rules),
}
diff --git a/components/script/dom/cssgroupingrule.rs b/components/script/dom/cssgroupingrule.rs
index 7204b8447f9..249aaccd42e 100644
--- a/components/script/dom/cssgroupingrule.rs
+++ b/components/script/dom/cssgroupingrule.rs
@@ -12,22 +12,21 @@ use dom::cssrule::CSSRule;
use dom::cssrulelist::{CSSRuleList, RulesSource};
use dom::cssstylesheet::CSSStyleSheet;
use dom_struct::dom_struct;
-use parking_lot::RwLock;
use std::sync::Arc;
-use style::shared_lock::SharedRwLock;
+use style::shared_lock::{SharedRwLock, Locked};
use style::stylesheets::CssRules as StyleCssRules;
#[dom_struct]
pub struct CSSGroupingRule {
cssrule: CSSRule,
#[ignore_heap_size_of = "Arc"]
- rules: Arc<RwLock<StyleCssRules>>,
+ rules: Arc<Locked<StyleCssRules>>,
rulelist: MutNullableJS<CSSRuleList>,
}
impl CSSGroupingRule {
pub fn new_inherited(parent_stylesheet: &CSSStyleSheet,
- rules: Arc<RwLock<StyleCssRules>>) -> CSSGroupingRule {
+ rules: Arc<Locked<StyleCssRules>>) -> CSSGroupingRule {
CSSGroupingRule {
cssrule: CSSRule::new_inherited(parent_stylesheet),
rules: rules,
diff --git a/components/script/dom/cssrulelist.rs b/components/script/dom/cssrulelist.rs
index 0b6351d237c..6321296c785 100644
--- a/components/script/dom/cssrulelist.rs
+++ b/components/script/dom/cssrulelist.rs
@@ -15,6 +15,7 @@ use dom::window::Window;
use dom_struct::dom_struct;
use parking_lot::RwLock;
use std::sync::Arc;
+use style::shared_lock::Locked;
use style::stylesheets::{CssRules, KeyframesRule, RulesMutateError};
#[allow(unsafe_code)]
@@ -43,7 +44,7 @@ pub struct CSSRuleList {
}
pub enum RulesSource {
- Rules(Arc<RwLock<CssRules>>),
+ Rules(Arc<Locked<CssRules>>),
Keyframes(Arc<RwLock<KeyframesRule>>),
}
@@ -52,7 +53,8 @@ impl CSSRuleList {
pub fn new_inherited(parent_stylesheet: &CSSStyleSheet, rules: RulesSource) -> CSSRuleList {
let dom_rules = match rules {
RulesSource::Rules(ref rules) => {
- rules.read().0.iter().map(|_| MutNullableJS::new(None)).collect()
+ let guard = parent_stylesheet.shared_lock().read();
+ rules.read_with(&guard).0.iter().map(|_| MutNullableJS::new(None)).collect()
}
RulesSource::Keyframes(ref rules) => {
rules.read().keyframes.iter().map(|_| MutNullableJS::new(None)).collect()
@@ -89,7 +91,9 @@ impl CSSRuleList {
let index = idx as usize;
let parent_stylesheet = self.parent_stylesheet.style_stylesheet();
- let new_rule = css_rules.write().insert_rule(rule, parent_stylesheet, index, nested)?;
+ let mut guard = parent_stylesheet.shared_lock.write();
+ let new_rule = css_rules.write_with(&mut guard)
+ .insert_rule(rule, parent_stylesheet, index, nested)?;
let parent_stylesheet = &*self.parent_stylesheet;
let dom_rule = CSSRule::new_specific(&window, parent_stylesheet, new_rule);
@@ -103,7 +107,8 @@ impl CSSRuleList {
match self.rules {
RulesSource::Rules(ref css_rules) => {
- css_rules.write().remove_rule(index)?;
+ let mut guard = self.parent_stylesheet.shared_lock().write();
+ css_rules.write_with(&mut guard).remove_rule(index)?;
let mut dom_rules = self.dom_rules.borrow_mut();
dom_rules[index].get().map(|r| r.detach());
dom_rules.remove(index);
@@ -133,9 +138,10 @@ impl CSSRuleList {
let parent_stylesheet = &self.parent_stylesheet;
match self.rules {
RulesSource::Rules(ref rules) => {
+ let guard = parent_stylesheet.shared_lock().read();
CSSRule::new_specific(self.global().as_window(),
parent_stylesheet,
- rules.read().0[idx as usize].clone())
+ rules.read_with(&guard).0[idx as usize].clone())
}
RulesSource::Keyframes(ref rules) => {
Root::upcast(CSSKeyframeRule::new(self.global().as_window(),
diff --git a/components/script/dom/cssstylesheet.rs b/components/script/dom/cssstylesheet.rs
index f8397a732c2..ee097313c2e 100644
--- a/components/script/dom/cssstylesheet.rs
+++ b/components/script/dom/cssstylesheet.rs
@@ -16,6 +16,7 @@ use dom::window::Window;
use dom_struct::dom_struct;
use std::cell::Cell;
use std::sync::Arc;
+use style::shared_lock::SharedRwLock;
use style::stylesheets::Stylesheet as StyleStyleSheet;
#[dom_struct]
@@ -72,6 +73,10 @@ impl CSSStyleSheet {
}
}
+ pub fn shared_lock(&self) -> &SharedRwLock {
+ &self.style_stylesheet.shared_lock
+ }
+
pub fn style_stylesheet(&self) -> &StyleStyleSheet {
&self.style_stylesheet
}
diff --git a/components/script/dom/htmlmetaelement.rs b/components/script/dom/htmlmetaelement.rs
index 2f78e74d3b4..a564c6b2661 100644
--- a/components/script/dom/htmlmetaelement.rs
+++ b/components/script/dom/htmlmetaelement.rs
@@ -101,8 +101,9 @@ impl HTMLMetaElement {
if let Some(translated_rule) = ViewportRule::from_meta(&**content) {
let document = self.upcast::<Node>().owner_doc();
let shared_lock = document.style_shared_lock();
+ let rule = CssRule::Viewport(Arc::new(RwLock::new(translated_rule)));
*self.stylesheet.borrow_mut() = Some(Arc::new(Stylesheet {
- rules: CssRules::new(vec![CssRule::Viewport(Arc::new(RwLock::new(translated_rule)))]),
+ rules: CssRules::new(vec![rule], shared_lock),
origin: Origin::Author,
shared_lock: shared_lock.clone(),
base_url: window_from_node(self).get_url(),
diff --git a/components/script/stylesheet_loader.rs b/components/script/stylesheet_loader.rs
index 26dffa5fe56..79b90be7426 100644
--- a/components/script/stylesheet_loader.rs
+++ b/components/script/stylesheet_loader.rs
@@ -175,10 +175,12 @@ impl FetchResponseListener for StylesheetContext {
}
StylesheetContextSource::Import(ref import) => {
let import = import.read();
+ let mut guard = document.style_shared_lock().write();
Stylesheet::update_from_bytes(&import.stylesheet,
&data,
protocol_encoding_label,
Some(environment_encoding),
+ &mut guard,
Some(&loader),
win.css_error_reporter(),
ParserContextExtraData::default());
diff --git a/components/style/encoding_support.rs b/components/style/encoding_support.rs
index 141291c896e..e485dc13cf0 100644
--- a/components/style/encoding_support.rs
+++ b/components/style/encoding_support.rs
@@ -12,7 +12,7 @@ use media_queries::MediaList;
use parser::ParserContextExtraData;
use self::encoding::{EncodingRef, DecoderTrap};
use servo_url::ServoUrl;
-use shared_lock::SharedRwLock;
+use shared_lock::{SharedRwLock, SharedRwLockWriteGuard};
use std::str;
use stylesheets::{Stylesheet, StylesheetLoader, Origin};
@@ -78,6 +78,7 @@ impl Stylesheet {
bytes: &[u8],
protocol_encoding_label: Option<&str>,
environment_encoding: Option<EncodingRef>,
+ guard: &mut SharedRwLockWriteGuard,
stylesheet_loader: Option<&StylesheetLoader>,
error_reporter: &ParseErrorReporter,
extra_data: ParserContextExtraData) {
@@ -85,6 +86,7 @@ impl Stylesheet {
bytes, protocol_encoding_label, environment_encoding);
Self::update_from_str(existing,
&string,
+ guard,
stylesheet_loader,
error_reporter,
extra_data)
diff --git a/components/style/gecko/arc_types.rs b/components/style/gecko/arc_types.rs
index daf0454c81e..798af00906e 100644
--- a/components/style/gecko/arc_types.rs
+++ b/components/style/gecko/arc_types.rs
@@ -39,7 +39,7 @@ macro_rules! impl_arc_ffi {
}
}
-impl_arc_ffi!(RwLock<CssRules> => ServoCssRules
+impl_arc_ffi!(Locked<CssRules> => ServoCssRules
[Servo_CssRules_AddRef, Servo_CssRules_Release]);
impl_arc_ffi!(Stylesheet => RawServoStyleSheet
diff --git a/components/style/stylesheets.rs b/components/style/stylesheets.rs
index 3f3d78c1e16..07dc2632b2d 100644
--- a/components/style/stylesheets.rs
+++ b/components/style/stylesheets.rs
@@ -21,7 +21,8 @@ use selector_parser::{SelectorImpl, SelectorParser};
use selectors::parser::SelectorList;
use servo_config::prefs::PREFS;
use servo_url::ServoUrl;
-use shared_lock::{SharedRwLock, Locked, SharedRwLockReadGuard, ToCssWithGuard};
+use shared_lock::{SharedRwLock, Locked, ToCssWithGuard};
+use shared_lock::{SharedRwLockReadGuard, SharedRwLockWriteGuard};
use std::cell::Cell;
use std::fmt;
use std::sync::Arc;
@@ -87,8 +88,8 @@ impl From<SingleRuleParseError> for RulesMutateError {
impl CssRules {
#[allow(missing_docs)]
- pub fn new(rules: Vec<CssRule>) -> Arc<RwLock<CssRules>> {
- Arc::new(RwLock::new(CssRules(rules)))
+ pub fn new(rules: Vec<CssRule>, shared_lock: &SharedRwLock) -> Arc<Locked<CssRules>> {
+ Arc::new(shared_lock.wrap(CssRules(rules)))
}
fn only_ns_or_import(&self) -> bool {
@@ -174,7 +175,7 @@ impl CssRules {
pub struct Stylesheet {
/// List of rules in the order they were found (important for
/// cascading order)
- pub rules: Arc<RwLock<CssRules>>,
+ pub rules: Arc<Locked<CssRules>>,
/// List of media associated with the Stylesheet.
pub media: Arc<Locked<MediaList>>,
/// The origin of this stylesheet.
@@ -302,7 +303,7 @@ impl CssRule {
CssRule::Import(ref lock) => {
let rule = lock.read();
let media = rule.stylesheet.media.read_with(guard);
- let rules = rule.stylesheet.rules.read();
+ let rules = rule.stylesheet.rules.read_with(guard);
// FIXME(emilio): Include the nested rules if the stylesheet is
// loaded.
f(&rules.0, Some(&media))
@@ -317,14 +318,14 @@ impl CssRule {
CssRule::Media(ref lock) => {
let media_rule = lock.read();
let mq = media_rule.media_queries.read_with(guard);
- let rules = &media_rule.rules.read().0;
+ let rules = &media_rule.rules.read_with(guard).0;
f(rules, Some(&mq))
}
CssRule::Supports(ref lock) => {
let supports_rule = lock.read();
let enabled = supports_rule.enabled;
if enabled {
- let rules = &supports_rule.rules.read().0;
+ let rules = &supports_rule.rules.read_with(guard).0;
f(rules, None)
} else {
f(&[], None)
@@ -478,7 +479,7 @@ impl ToCssWithGuard for KeyframesRule {
#[derive(Debug)]
pub struct MediaRule {
pub media_queries: Arc<Locked<MediaList>>,
- pub rules: Arc<RwLock<CssRules>>,
+ pub rules: Arc<Locked<CssRules>>,
}
impl ToCssWithGuard for MediaRule {
@@ -489,7 +490,7 @@ impl ToCssWithGuard for MediaRule {
try!(dest.write_str("@media "));
try!(self.media_queries.read_with(guard).to_css(dest));
try!(dest.write_str(" {"));
- for rule in self.rules.read().0.iter() {
+ for rule in self.rules.read_with(guard).0.iter() {
try!(dest.write_str(" "));
try!(rule.to_css(guard, dest));
}
@@ -504,7 +505,7 @@ pub struct SupportsRule {
/// The parsed condition
pub condition: SupportsCondition,
/// Child rules
- pub rules: Arc<RwLock<CssRules>>,
+ pub rules: Arc<Locked<CssRules>>,
/// The result of evaluating the condition
pub enabled: bool,
}
@@ -515,7 +516,7 @@ impl ToCssWithGuard for SupportsRule {
try!(dest.write_str("@supports "));
try!(self.condition.to_css(dest));
try!(dest.write_str(" {"));
- for rule in self.rules.read().0.iter() {
+ for rule in self.rules.read_with(guard).0.iter() {
try!(dest.write_str(" "));
try!(rule.to_css(guard, dest));
}
@@ -555,10 +556,11 @@ impl Stylesheet {
/// Updates an empty stylesheet from a given string of text.
pub fn update_from_str(existing: &Stylesheet,
css: &str,
+ guard: &mut SharedRwLockWriteGuard,
stylesheet_loader: Option<&StylesheetLoader>,
error_reporter: &ParseErrorReporter,
extra_data: ParserContextExtraData) {
- let mut rules = existing.rules.write();
+ let mut rules = existing.rules.write_with(guard);
let mut namespaces = existing.namespaces.write();
assert!(rules.is_empty());
@@ -613,18 +615,22 @@ impl Stylesheet {
origin: origin,
base_url: base_url,
namespaces: RwLock::new(Namespaces::default()),
- rules: CssRules::new(vec![]),
+ rules: CssRules::new(Vec::new(), &shared_lock),
media: Arc::new(shared_lock.wrap(media)),
- shared_lock: shared_lock.clone(),
+ shared_lock: shared_lock,
dirty_on_viewport_size_change: AtomicBool::new(false),
disabled: AtomicBool::new(false),
};
- Self::update_from_str(&s,
- css,
- stylesheet_loader,
- error_reporter,
- extra_data);
+ {
+ let mut guard = s.shared_lock.write();
+ Self::update_from_str(&s,
+ css,
+ &mut guard,
+ stylesheet_loader,
+ error_reporter,
+ extra_data);
+ }
s
}
@@ -666,7 +672,7 @@ impl Stylesheet {
#[inline]
pub fn effective_rules<F>(&self, device: &Device, guard: &SharedRwLockReadGuard, mut f: F)
where F: FnMut(&CssRule) {
- effective_rules(&self.rules.read().0, device, guard, &mut f);
+ effective_rules(&self.rules.read_with(guard).0, device, guard, &mut f);
}
/// Returns whether the stylesheet has been explicitly disabled through the
@@ -809,7 +815,7 @@ impl<'a> AtRuleParser for TopLevelRuleParser<'a> {
ImportRule {
url: url,
stylesheet: Arc::new(Stylesheet {
- rules: Arc::new(RwLock::new(CssRules(vec![]))),
+ rules: CssRules::new(Vec::new(), self.shared_lock),
media: media,
shared_lock: self.shared_lock.clone(),
origin: self.context.stylesheet_origin,
@@ -907,7 +913,7 @@ struct NestedRuleParser<'a, 'b: 'a> {
}
impl<'a, 'b> NestedRuleParser<'a, 'b> {
- fn parse_nested_rules(&self, input: &mut Parser) -> Arc<RwLock<CssRules>> {
+ fn parse_nested_rules(&self, input: &mut Parser) -> Arc<Locked<CssRules>> {
let mut iter = RuleListParser::new_for_nested_rule(input, self.clone());
let mut rules = Vec::new();
while let Some(result) = iter.next() {
@@ -920,7 +926,7 @@ impl<'a, 'b> NestedRuleParser<'a, 'b> {
}
}
}
- CssRules::new(rules)
+ CssRules::new(rules, self.shared_lock)
}
}
diff --git a/components/style/stylist.rs b/components/style/stylist.rs
index 0ff9d20d883..73684a4b8fd 100644
--- a/components/style/stylist.rs
+++ b/components/style/stylist.rs
@@ -502,7 +502,7 @@ impl Stylist {
return true
}
- mq_eval_changed(guard, &stylesheet.rules.read().0, &self.device, &device)
+ mq_eval_changed(guard, &stylesheet.rules.read_with(guard).0, &self.device, &device)
});
self.device = Arc::new(device);
diff --git a/ports/geckolib/glue.rs b/ports/geckolib/glue.rs
index e0335ee0005..e260f365fa5 100644
--- a/ports/geckolib/glue.rs
+++ b/ports/geckolib/glue.rs
@@ -402,6 +402,8 @@ pub extern "C" fn Servo_StyleSheet_ClearAndUpdate(stylesheet: RawServoStyleSheet
referrer: *mut ThreadSafeURIHolder,
principal: *mut ThreadSafePrincipalHolder)
{
+ let global_style_data = &*GLOBAL_STYLE_DATA;
+ let mut guard = global_style_data.shared_lock.write();
let input = unsafe { data.as_ref().unwrap().as_str_unchecked() };
let extra_data = unsafe { ParserContextExtraData {
base: Some(GeckoArcURI::new(base)),
@@ -422,9 +424,9 @@ pub extern "C" fn Servo_StyleSheet_ClearAndUpdate(stylesheet: RawServoStyleSheet
};
let sheet = Stylesheet::as_arc(&stylesheet);
- sheet.rules.write().0.clear();
+ sheet.rules.write_with(&mut guard).0.clear();
- Stylesheet::update_from_str(&sheet, input, loader,
+ Stylesheet::update_from_str(&sheet, input, &mut guard, loader,
&StdoutErrorReporter, extra_data);
}
@@ -510,7 +512,9 @@ pub extern "C" fn Servo_StyleSet_NoteStyleSheetsChanged(raw_data: RawServoStyleS
#[no_mangle]
pub extern "C" fn Servo_StyleSheet_HasRules(raw_sheet: RawServoStyleSheetBorrowed) -> bool {
- !Stylesheet::as_arc(&raw_sheet).rules.read().0.is_empty()
+ let global_style_data = &*GLOBAL_STYLE_DATA;
+ let guard = global_style_data.shared_lock.read();
+ !Stylesheet::as_arc(&raw_sheet).rules.read_with(&guard).0.is_empty()
}
#[no_mangle]
@@ -521,7 +525,9 @@ pub extern "C" fn Servo_StyleSheet_GetRules(sheet: RawServoStyleSheetBorrowed) -
#[no_mangle]
pub extern "C" fn Servo_CssRules_ListTypes(rules: ServoCssRulesBorrowed,
result: nsTArrayBorrowed_uintptr_t) {
- let rules = RwLock::<CssRules>::as_arc(&rules).read();
+ let global_style_data = &*GLOBAL_STYLE_DATA;
+ let guard = global_style_data.shared_lock.read();
+ let rules = Locked::<CssRules>::as_arc(&rules).read_with(&guard);
let iter = rules.0.iter().map(|rule| rule.rule_type() as usize);
let (size, upper) = iter.size_hint();
debug_assert_eq!(size, upper.unwrap());
@@ -533,10 +539,12 @@ pub extern "C" fn Servo_CssRules_ListTypes(rules: ServoCssRulesBorrowed,
pub extern "C" fn Servo_CssRules_InsertRule(rules: ServoCssRulesBorrowed, sheet: RawServoStyleSheetBorrowed,
rule: *const nsACString, index: u32, nested: bool,
rule_type: *mut u16) -> nsresult {
- let rules = RwLock::<CssRules>::as_arc(&rules);
+ let global_style_data = &*GLOBAL_STYLE_DATA;
+ let mut guard = global_style_data.shared_lock.write();
+ let rules = Locked::<CssRules>::as_arc(&rules);
let sheet = Stylesheet::as_arc(&sheet);
let rule = unsafe { rule.as_ref().unwrap().as_str_unchecked() };
- match rules.write().insert_rule(rule, sheet, index as usize, nested) {
+ match rules.write_with(&mut guard).insert_rule(rule, sheet, index as usize, nested) {
Ok(new_rule) => {
*unsafe { rule_type.as_mut().unwrap() } = new_rule.rule_type() as u16;
nsresult::NS_OK
@@ -547,8 +555,10 @@ pub extern "C" fn Servo_CssRules_InsertRule(rules: ServoCssRulesBorrowed, sheet:
#[no_mangle]
pub extern "C" fn Servo_CssRules_DeleteRule(rules: ServoCssRulesBorrowed, index: u32) -> nsresult {
- let rules = RwLock::<CssRules>::as_arc(&rules);
- match rules.write().remove_rule(index as usize) {
+ let global_style_data = &*GLOBAL_STYLE_DATA;
+ let mut guard = global_style_data.shared_lock.write();
+ let rules = Locked::<CssRules>::as_arc(&rules);
+ match rules.write_with(&mut guard).remove_rule(index as usize) {
Ok(_) => nsresult::NS_OK,
Err(err) => err.into()
}
@@ -562,7 +572,9 @@ macro_rules! impl_basic_rule_funcs {
} => {
#[no_mangle]
pub extern "C" fn $getter(rules: ServoCssRulesBorrowed, index: u32) -> Strong<$raw_type> {
- let rules = RwLock::<CssRules>::as_arc(&rules).read();
+ let global_style_data = &*GLOBAL_STYLE_DATA;
+ let guard = global_style_data.shared_lock.read();
+ let rules = Locked::<CssRules>::as_arc(&rules).read_with(&guard);
match rules.0[index as usize] {
CssRule::$name(ref rule) => rule.clone().into_strong(),
_ => {
diff --git a/tests/unit/style/media_queries.rs b/tests/unit/style/media_queries.rs
index 652d68ce953..3d2f2809a63 100644
--- a/tests/unit/style/media_queries.rs
+++ b/tests/unit/style/media_queries.rs
@@ -35,7 +35,7 @@ fn test_media_rule<F>(css: &str, callback: F)
ParserContextExtraData::default());
let mut rule_count = 0;
let guard = stylesheet.shared_lock.read();
- media_queries(&guard, &stylesheet.rules.read().0, &mut |mq| {
+ media_queries(&guard, &stylesheet.rules.read_with(&guard).0, &mut |mq| {
rule_count += 1;
callback(mq, css);
});
diff --git a/tests/unit/style/rule_tree/bench.rs b/tests/unit/style/rule_tree/bench.rs
index 623a534159a..aeac93912aa 100644
--- a/tests/unit/style/rule_tree/bench.rs
+++ b/tests/unit/style/rule_tree/bench.rs
@@ -49,7 +49,8 @@ fn parse_rules(css: &str) -> Vec<(StyleSource, CascadeLevel)> {
None,
&ErrorringErrorReporter,
ParserContextExtraData {});
- let rules = s.rules.read();
+ let guard = s.shared_lock.read();
+ let rules = s.rules.read_with(&guard);
rules.0.iter().filter_map(|rule| {
match *rule {
CssRule::Style(ref style_rule) => Some(style_rule),
diff --git a/tests/unit/style/stylesheets.rs b/tests/unit/style/stylesheets.rs
index 590e93185f4..00cefe23ce9 100644
--- a/tests/unit/style/stylesheets.rs
+++ b/tests/unit/style/stylesheets.rs
@@ -263,7 +263,7 @@ fn test_parse_stylesheet() {
]
})))
- ]),
+ ], &stylesheet.shared_lock),
};
assert_eq!(format!("{:#?}", stylesheet), format!("{:#?}", expected));
generated by cgit v1.2.3 (git 2.25.1) at 2025-04-28 06:12:02 +0000