Auto merge of #15837 - upsuper:bug1344135, r=emilio

Return true in set_property only when declaration block is changed

This is [bug 1344135](https://bugzilla.mozilla.org/show_bug.cgi?id=1344135).

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/15837)
<!-- Reviewable:end -->
This commit is contained in:
bors-servo 2017-03-07 13:51:33 -08:00 committed by GitHub
commit b11847d86c
3 changed files with 13 additions and 10 deletions

View file

@ -837,10 +837,11 @@ fn set_property(declarations: RawServoDeclarationBlockBorrowed, property_id: Pro
Box::new(StdoutErrorReporter), extra_data) {
let mut declarations = RwLock::<PropertyDeclarationBlock>::as_arc(&declarations).write();
let importance = if is_important { Importance::Important } else { Importance::Normal };
let mut changed = false;
for decl in decls.into_iter() {
declarations.set_parsed_declaration(decl.0, importance);
changed |= declarations.set_parsed_declaration(decl.0, importance);
}
true
changed
} else {
false
}