Bug 1348273 - Convert the layout annotation; r?dholbert draft
authorGabriele Svelto <gsvelto@mozilla.com>
Sat, 17 Mar 2018 22:23:41 +0100
changeset 781406 ce25480d8082a2c92664b26cc1c7f29fa39e30f8
parent 781405 ec23ba7d43633ea9fca4af4c73e596c044ebdf1a
child 781407 37318fe37ce2c4e017164fdce8d53f14d2d2ac19
push id106292
push usergsvelto@mozilla.com
push dateThu, 12 Apr 2018 22:06:23 +0000
reviewersdholbert
bugs1348273
milestone61.0a1
Bug 1348273 - Convert the layout annotation; r?dholbert MozReview-Commit-ID: DGjl1lELte5
layout/style/nsLayoutStylesheetCache.cpp
toolkit/crashreporter/CrashAnnotations.yaml
--- a/layout/style/nsLayoutStylesheetCache.cpp
+++ b/layout/style/nsLayoutStylesheetCache.cpp
@@ -741,18 +741,19 @@ AnnotateCrashReport(nsIURI* aURI)
         }
       }
       delete find;
     }
   } else {
     annotation.AppendLiteral("No GRE omnijar\n");
   }
 
-  CrashReporter::AnnotateCrashReport(NS_LITERAL_CSTRING("SheetLoadFailure"),
-                                     NS_ConvertUTF16toUTF8(annotation));
+  CrashReporter::AnnotateCrashReport(
+    CrashReporter::Annotation::SheetLoadFailure,
+    NS_ConvertUTF16toUTF8(annotation));
 }
 
 static void
 ErrorLoadingSheet(nsIURI* aURI, const char* aMsg, FailureAction aFailureAction)
 {
   nsPrintfCString errorMessage("%s loading built-in stylesheet '%s'",
                                aMsg,
                                aURI ? aURI->GetSpecOrDefault().get() : "");
--- a/toolkit/crashreporter/CrashAnnotations.yaml
+++ b/toolkit/crashreporter/CrashAnnotations.yaml
@@ -509,16 +509,22 @@ SecondsSinceLastCrash:
   type: string
   ping: true
 
 ServerURL:
   description: >
     URL used to post the crash report.
   type: string
 
+SheetLoadFailure:
+  description: >
+    Set when failing to load a built-in style sheet. This can contain a
+    potentially very large amount of diagnostic information.
+  type: string
+
 StartupTime:
   description: >
     The time when Firefox was launched expressed in seconds since the Epoch.
   type: integer
   content: false
 
 StatFailure:
   description: >