Backed out changeset b56791a96f96 (bug 1533424) for crashtest on test_parsingMode.html on a CLOSED TREE
authorOana Pop Rus <opoprus@mozilla.com>
Fri, 08 Mar 2019 17:42:37 +0200
changeset 521051 def4c84342462fb2b23a7a3940f7cfcad919dcd6
parent 521050 6d82d52f6279fe2307238d2e9a9d7fdfbe7c7bb1
child 521052 d3d719ba069153ae29a99f60ec537133345c7ff0
push id10862
push userffxbld-merge
push dateMon, 11 Mar 2019 13:01:11 +0000
treeherdermozilla-beta@a2e7f5c935da [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1533424
milestone67.0a1
backs outb56791a96f96525d2d4fbec907f6c7c58a9e3dcc
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Backed out changeset b56791a96f96 (bug 1533424) for crashtest on test_parsingMode.html on a CLOSED TREE
layout/style/StyleSheet.cpp
--- a/layout/style/StyleSheet.cpp
+++ b/layout/style/StyleSheet.cpp
@@ -962,23 +962,16 @@ void StyleSheet::FinishParse() {
   SetSourceURL(sourceURL);
 }
 
 nsresult StyleSheet::ReparseSheet(const nsAString& aInput) {
   if (!IsComplete()) {
     return NS_ERROR_DOM_INVALID_ACCESS_ERR;
   }
 
-  // Allowing to modify UA sheets is dangerous (in the sense that C++ code
-  // relies on rules in those sheets), plus they're probably going to be shared
-  // across processes in which case this is directly a no-go.
-  if (GetOrigin() == OriginFlags::UserAgent) {
-    return NS_ERROR_DOM_NO_MODIFICATION_ALLOWED_ERR;
-  }
-
   // Hold strong ref to the CSSLoader in case the document update
   // kills the document
   RefPtr<css::Loader> loader;
   if (Document* doc = GetAssociatedDocument()) {
     loader = doc->CSSLoader();
     NS_ASSERTION(loader, "Document with no CSS loader!");
   } else {
     loader = new css::Loader;