Mail: Tweak the height of the menu fields

Since hrev45725 we no longer resize fixed sized menu fields to their preferred
height in the constructor so as to be backwards compatible with the behavior on
BeOS R5. As a consequence though, the menu fields in Mail are now a bit too
tall so this commit tweaks the menu fields to be just a bit shorter matching
the height of the text controls.
BColorControl: Style fixes

* Update copyright in header, add my name, alphabetize.
* Rename _ColorRamp() to _DrawColorRamp().
* Rename update parameter to updateRect
BColorControl: Create _SetCellSize() method

...eliminating duplicated code.
BColorControl: Size text rect based on font size

* Also change kMinCellSize from a uint32 to a float so that it can be used
  with std::min() and std::max() instead of min_c() and max_c().
* Set the text controls sizes and margins based on the font size. Also rework
  _TextRectOffset() so that it will get the right spacing from by dividing the
  palette frame by 3.
* Replace bare numbers and refactor with calculation or magic constant.
* Create a private method _TextRectOffset() which calculates and
  returns the vertical text rect offset to use based on the font size.
* Replace 2.0 with new kBevelSpacing constant where appropriate.
* fPaletteFrame calculation in _LayoutView() was refactored but should
  not have changed.
BColorControl: Rename bevelRect to just rect

... with an explanatory comment. Correct similar comment.
Tiny style fix.
BColorControl: Introduce a _PaletteFrame() method

... eliminating duplicate code.
BColorControl: Introduce kRampCount magic constant.
BColorControl: Refactor _LayoutView()

In a few cases we were doing the work of _LayoutView() twice because
we called _LayoutView() and then called ResizeToPreferred() which called
_LayoutView() again.

Now only call ResizeToPreferred() which calls _LayoutView().
BColorControl: Position text controls better

Position the text control vertically in the middle of the ramp in
the case that the text controls are pushed down so that the red, green,
and blue text controls are next to the red, green, and blue ramps.
Check if disk has compilation artist

... before looking for track specific artist information. Fixes #9813

Signed-off-by: John Scipione <jscipione@gmail.com>
nfs4: Clear PeerAddress before attempting to get local address

If PeerAddress is cleared before invoking GetLocalAddress() we make sure
it won't contain any garbage data in a very unlike case when getsockname()
fails.
nfs4: Fix CID #991617: Set cookie file system at its creation
nfs4: Remove unused member variable NFS4Server::fClientIdInit
nfs4: Fix CID #1032280: Prevent integer overflow

 * Do not increase delay after tenth attempt
 * Cast 1 to bigtime_t before shifting it left
nfs4: Fix CID #1032257: ERR_DENIED is expected only when a cookie is given
nfs4: Fix CID #1032256: Use 4kB block size if server reports invalid value
nfs4: Use nothrow new when creating CallbackRequest
nfs4: Use nothrow new when creating LockInfo
nfs4: Fix CID #991493: Release memory when returing an error code
BFont docs: Clarify escapement_delta language.

Make it clear that the values provided by the escapement_delta struct are an
input to App Server which allows the user to specify extra horizontal space around each character and is not an output provided by App Server.