Bug 906116 - part2: Fix comment for nsChangeHint_ReconstructFrame. draft
authorJeremy Chen <jeremychen@mozilla.com>
Wed, 06 Jul 2016 13:06:13 +0800
changeset 384356 bac9c081aa890e057a9608ff8b18be5528a370fe
parent 384355 fc53e4e146b1bf463c172aff73e68e78fe9d017d
child 384357 e99fe986ba002ee6a16417d47dab25b25b6c96fe
push id22251
push userjichen@mozilla.com
push dateWed, 06 Jul 2016 05:07:25 +0000
bugs906116
milestone50.0a1
Bug 906116 - part2: Fix comment for nsChangeHint_ReconstructFrame. The comment for nsChangeHint_ReconstructFrame is out-of-date. In RestyleManager::ProcessRestyledFrames, we actually ignore all of the other hints if ReconstructFrame is set. The old version was written when ReconstructFrame was listed last. So, update the comment. MozReview-Commit-ID: 97wMrW6S6ID
layout/base/nsChangeHint.h
--- a/layout/base/nsChangeHint.h
+++ b/layout/base/nsChangeHint.h
@@ -65,18 +65,21 @@ enum nsChangeHint {
   /**
    * Updates all descendants. Any placeholder descendants' out-of-flows
    * are also descendants of the transformed frame, so they're updated.
    */
   nsChangeHint_UpdateTransformLayer = 1 << 9,
 
   /**
    * Change requires frame change (e.g., display:).
-   * This subsumes all the above. Reconstructs all frame descendants,
-   * including following placeholders to out-of-flows.
+   * Reconstructs all frame descendants, including following placeholders
+   * to out-of-flows.
+   *
+   * Note that this subsumes all the other change hints. (see
+   * RestyleManager::ProcessRestyledFrames for details).
    */
   nsChangeHint_ReconstructFrame = 1 << 10,
 
   /**
    * The frame's overflow area has changed. Does not update any descendant
    * frames.
    */
   nsChangeHint_UpdateOverflow = 1 << 11,