From 1edf5571f6863e9ab08172dc611b5a167dc01836 Mon Sep 17 00:00:00 2001 From: Ting-Yu Lin Date: Thu, 12 Mar 2020 21:50:18 +0000 Subject: [PATCH] Bug 1622008 Part 1 - Revise nsFrameManager's documentation. r=dholbert Differential Revision: https://phabricator.services.mozilla.com/D66598 --HG-- extra : moz-landing-system : lando --- layout/base/nsFrameManager.h | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/layout/base/nsFrameManager.h b/layout/base/nsFrameManager.h index 1ed0e1e24732..c27eddf0a5e1 100644 --- a/layout/base/nsFrameManager.h +++ b/layout/base/nsFrameManager.h @@ -4,7 +4,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -/* storage of the frame tree and information about it */ +/* Owns the frame tree and provides APIs to manipulate it */ #ifndef _nsFrameManager_h_ #define _nsFrameManager_h_ @@ -24,13 +24,10 @@ class PresShell; } // namespace mozilla /** - * Frame manager interface. The frame manager serves one purpose: - *
  • handles structural modifications to the frame model. If the frame model - * lock can be acquired, then the changes are processed immediately; otherwise, - * they're queued and processed later. - * - * FIXME(emilio): The comment above doesn't make any sense, there's no "frame - * model lock" of any sort afaict. + * Frame manager interface. The frame manager owns the frame tree model, and + * handles structural manipulations to it, such as appending and inserting a + * list of frames to a parent frame, or removing a child frame from a parent + * frame. */ class nsFrameManager { typedef mozilla::PresShell PresShell;