Bug 1265591 patch 3 - Make the comment describing RestyleForCSSRuleChanges match reality. r=heycam
authorL. David Baron <dbaron@dbaron.org>
Tue, 19 Apr 2016 11:38:35 -0700
changeset 317698 aea48706d499202a9d2b9b6bfd2292cc08fe1535
parent 317697 f411f50ad23b194b570f2e30fe6b06b8881b183f
child 317699 68a682ef9e59eabb5d0f1fda05e84a773ca8340b
push id9480
push userjlund@mozilla.com
push dateMon, 25 Apr 2016 17:12:58 +0000
treeherdermozilla-aurora@0d6a91c76a9e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersheycam
bugs1265591
milestone48.0a1
Bug 1265591 patch 3 - Make the comment describing RestyleForCSSRuleChanges match reality. r=heycam MozReview-Commit-ID: 9CT2k59cYP2
layout/base/nsIPresShell.h
--- a/layout/base/nsIPresShell.h
+++ b/layout/base/nsIPresShell.h
@@ -353,21 +353,30 @@ public:
    * author level of the cascade, including the HTML preshint level.
    */
   // XXX these could easily be inlined, but there is a circular #include
   // problem with nsStyleSet.
   void SetAuthorStyleDisabled(bool aDisabled);
   bool GetAuthorStyleDisabled() const;
 
   /*
-   * Called when stylesheets are added/removed/enabled/disabled to rebuild
-   * all style data for a given pres shell without necessarily reconstructing
-   * all of the frames.  This will not reconstruct style synchronously; if
-   * you need to do that, call FlushPendingNotifications to flush out style
-   * reresolves.
+   * Called when stylesheets are added/removed/enabled/disabled to
+   * recompute style and clear other cached data as needed.  This will
+   * not reconstruct style synchronously; if you need to do that, call
+   * FlushPendingNotifications to flush out style reresolves.
+   *
+   * This handles the the addition and removal of the various types of
+   * style rules that can be in CSS style sheets, such as @font-face
+   * rules and @counter-style rules.
+   *
+   * It requires that StyleSheetAdded, StyleSheetRemoved,
+   * StyleSheetApplicableStateChanged, StyleRuleAdded, StyleRuleRemoved,
+   * or StyleRuleChanged has been called on the style sheets that have
+   * changed.
+   *
    * // XXXbz why do we have this on the interface anyway?  The only consumer
    * is calling AddOverrideStyleSheet/RemoveOverrideStyleSheet, and I think
    * those should just handle reconstructing style data...
    */
   void RestyleForCSSRuleChanges();
 
   /**
    * Update the style set somehow to take into account changed prefs which