As a security precaution, we have turned on the setting "Require API key authentication for API requests" for everyone. If this has broken something, please contact
Last Comment Bug 746285 - pack nsXBLBinding more carefully for 64-bit
: pack nsXBLBinding more carefully for 64-bit
Product: Core
Classification: Components
Component: XBL (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla15
Assigned To: Nathan Froyd [:froydnj]
: Andrew Overholt [:overholt]
Depends on:
  Show dependency treegraph
Reported: 2012-04-17 12:39 PDT by Nathan Froyd [:froydnj]
Modified: 2012-04-24 18:05 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (2.06 KB, patch)
2012-04-17 12:39 PDT, Nathan Froyd [:froydnj]
bzbarsky: review+
Details | Diff | Splinter Review

Description User image Nathan Froyd [:froydnj] 2012-04-17 12:39:48 PDT
Created attachment 615828 [details] [diff] [review]

nsXBLBinding is laid out like so:

  int mRefCnt;
  nsIContent *mBoundElement;
  nsClassHashtable* mInsertionPointTable;
  bool mIsStyleBinding;
  bool mMarkedForDeath;

which introduces padding after mRefCnt and at the end of the structure.  Moving the booleans up into the padding required after mRefCnt saves a word at the end of the structure.
Comment 1 User image Boris Zbarsky [:bz] (still a bit busy) 2012-04-17 12:43:26 PDT
Comment on attachment 615828 [details] [diff] [review]

Comment 2 User image Nathan Froyd [:froydnj] 2012-04-20 13:25:20 PDT

Note You need to log in before you can comment on or make changes to this bug.