Bug 509380 - Optimize nsPeekOffsetStruct for memory usage; r+sr=roc
authorArpad Borsos <arpad.borsos@googlemail.com>
Mon, 10 Aug 2009 09:47:11 +0200
changeset 31430 707834b4b7def619d9acbdd611d3241df1ace0fc
parent 31429 c2cc58728c8bcd0820e360ec99c075d37c791346
child 31431 b50beb2bef4774a8b5c1f276c88600eff4db4de0
push id1
push usersledru@mozilla.com
push dateThu, 04 Dec 2014 17:57:20 +0000
bugs509380
milestone1.9.2a2pre
Bug 509380 - Optimize nsPeekOffsetStruct for memory usage; r+sr=roc
layout/generic/nsFrameSelection.h
--- a/layout/generic/nsFrameSelection.h
+++ b/layout/generic/nsFrameSelection.h
@@ -136,50 +136,50 @@ struct NS_STACK_CLASS nsPeekOffsetStruct
   // mStartOffset: Offset into the content of the current frame where the peek starts.
   //               Used with: eSelectCharacter, eSelectWord
   PRInt32 mStartOffset;
   
   // mDesiredX: The desired x coordinate for the caret.
   //            Used with: eSelectLine.
   nscoord mDesiredX;
 
-  // mJumpLines: Whether to allow jumping across line boundaries.
-  //             Used with: eSelectCharacter, eSelectWord.
-  PRBool mJumpLines;
-
-  // mScrollViewStop: Whether to stop when reaching a scroll view boundary.
-  //                  Used with: eSelectCharacter, eSelectWord, eSelectLine.
-  PRBool mScrollViewStop;
-
-  // mIsKeyboardSelect: Whether the peeking is done in response to a keyboard action.
-  //                    Used with: eSelectWord.
-  PRBool mIsKeyboardSelect;
-
-  // mVisual: Whether bidi caret behavior is visual (PR_TRUE) or logical (PR_FALSE).
-  //          Used with: eSelectCharacter, eSelectWord, eSelectBeginLine, eSelectEndLine.
-  PRBool mVisual;
-
   // mWordMovementType: An enum that determines whether to prefer the start or end of a word
   //                    or to use the default beahvior, which is a combination of 
   //                    direction and the platform-based pref
   //                    "layout.word_select.eat_space_to_next_word"
   EWordMovementType mWordMovementType;
 
+  // mJumpLines: Whether to allow jumping across line boundaries.
+  //             Used with: eSelectCharacter, eSelectWord.
+  PRPackedBool mJumpLines;
+
+  // mScrollViewStop: Whether to stop when reaching a scroll view boundary.
+  //                  Used with: eSelectCharacter, eSelectWord, eSelectLine.
+  PRPackedBool mScrollViewStop;
+
+  // mIsKeyboardSelect: Whether the peeking is done in response to a keyboard action.
+  //                    Used with: eSelectWord.
+  PRPackedBool mIsKeyboardSelect;
+
+  // mVisual: Whether bidi caret behavior is visual (PR_TRUE) or logical (PR_FALSE).
+  //          Used with: eSelectCharacter, eSelectWord, eSelectBeginLine, eSelectEndLine.
+  PRPackedBool mVisual;
+
   /*** Output arguments ***/
 
   // mResultContent: Content reached as a result of the peek.
   nsCOMPtr<nsIContent> mResultContent;
 
-  // mContentOffset: Offset into content reached as a result of the peek.
-  PRInt32 mContentOffset;
-
   // mResultFrame: Frame reached as a result of the peek.
   //               Used with: eSelectCharacter, eSelectWord.
   nsIFrame *mResultFrame;
 
+  // mContentOffset: Offset into content reached as a result of the peek.
+  PRInt32 mContentOffset;
+
   // mAttachForward: When the result position is between two frames,
   //                 indicates which of the two frames the caret should be painted in.
   //                 PR_FALSE means "the end of the frame logically before the caret", 
   //                 PR_TRUE means "the beginning of the frame logically after the caret".
   //                 Used with: eSelectLine, eSelectBeginLine, eSelectEndLine.
   PRBool mAttachForward;
 };