Various Cocoa improvements and fixes

Changed behaviour of vtkCocoaRenderWindowInteractor::TerminateApp().
It used to do one of two things: 1) terminate the application completely,
or 2) close the NSWindow and stop the event loop.  Now it never closes
the NSWindow, never terminates the application, and always stops the
event loop.  Start() and TerminateApp() are now pairs, which is how other
platforms already behave.  Also refactored NSApplication runloop
starting/stopping code into static functions that do only that.  Makes
more sense since there is only one event loop, there doesn't need to be
an association with any class or instance.  Thanks to Pat Marion for
reporting this problem and creating a patch.

Changed of behaviour observing for NSWindow closing.  First, moved the
private vtkCocoaServer class from vtkCocoaRenderWindowInteractor to
vtkCocoaRenderWindow.  Its job is to observe window closing.  Since
vtkCocoaRenderWindow creates the NSWindow, it make more sense there.
We were previously careful to only observe NSWindows created by VTK,
but are now also careful to *stop* observing only those same windows,
whereas previously there was an outside chance of stopping observation
of all window closes, which is something a library should not do
(who knows what else the application is doing).  Now when an observed
NSWindow closes, we no longer stop the event loop, but merely remove
our reference to it.

Changed vtkCocoaRenderWindow::SetWindowName() to interpret its char*
string parameter as UTF8 instead of ASCII.

Fixed possible memory leaks in ~vtkCocoaRenderWindow.

Replaced most invocations of 'autorelease' in favour of 'release'.
The motivation being that VTK, as a library, might sometimes be called
before an autorelease pool is even setup, so better to avoid autoreleased
objects as much as possible.

Fixed some whitespace and coding conventions
(asterisk positioning, constness, etc.)

Change-Id: I5a5d410012980ee9f6c637c340e98e3daa5aee74
make rubber band look correct on scenes with textures

vtkTexture turns on an alpha test. If your scene has any textures in
it the selection rubber band looks wrong. The reason is that when
rubber band draws into the screen with renderwindow::SetRGBA the background
pixels, presumably with failing alpha values, can not change.

Change-Id: I6c78592234b9b4e38e020f148def9d84d7d66bf8
Merge topic 'fix-rubber-band' into master

252798f0 make rubber band look correct on scenes with textures
Add sorting capabilities

Change-Id: I9dfd374b20c9516c2f2bdf84a82dbb8b90365c1f
Merge topic 'catalyst-web-composite' into master

4ef82649 Add sorting capabilities
In wrappers, guard against #including dirs.

The wrappers tools try to read all #included headers in order to get all
the preprocessor macros that they might need to properly parse the code.
However, if a header cannot be loaded, they are supposed to be ignored.
David DeMarle recently discovered that if the #include directive found
a directory with the name of the header that it was looking for, then the
wrapper tools would fail.  I've added an "isdir" check to guard against
this.

Change-Id: I4847b8806b3e28a42aa299dce0dfe58254c20084
Return early in TransformPoints if input is NULL.

Change-Id: I334f78b9429dfec78f758f10f8fc30f439d8c276
Merge topic '14615-fix-mpi-segfault' into master

e52eff07 Return early in TransformPoints if input is NULL.
Netcdf now handles different directories for c and cxx bindings.

Change-Id: I50f463046399ec1c6186ffe0b92066a642e354f9
Refresh workbench UI and workflow

Change-Id: I3bf5b6ce78c9653d094e9b3cbb1302c4ac498990
Merge topic 'catalyst-web-composite' into master

eaf37306 Refresh workbench UI and workflow
Merge topic 'preproc-dir-check' into master

c8c78cd6 In wrappers, guard against #including dirs.
Fix Composite Javascript for WebKit

Change-Id: I5da656305be57dd9d8e78c6f498daecdab2120f8
Merge topic 'catalyst-web-composite' into master

8f3a3574 Fix Composite Javascript for WebKit
Merge topic 'find_netcdf_support_multiple_include_dirs' into master

b7cb56bd Netcdf now handles different directories for c and cxx bindings.
Fix cell picker for images with negative spacing.

If the spacing of the image data was negative in any of the three
dimensions, it resulted in a failed bounding box test for the image
in the vtkCellPicker.  This patch adds a check for negative spacing.

Change-Id: Ia6b7c31fa78988900ce2e68f0ee8cd9bc8ecbf8d
Merge topic 'cell-picker-image-spacing' into master

8eb00d2f Fix cell picker for images with negative spacing.
Update system font test baseline.

Change-Id: I1a51d9f1c44ce55863077bd2dfba8deacc53f26b
Merge topic 'update-baseline' into master

67ee41e8 Update system font test baseline.
Fix uninitialized array warnings.

There were dashboard warnings about array elements that might be
used uninitialized.  Set the arrays to reasonable default values.

Change-Id: I6507d778ff315367c2798b1568e2b4eac7731cd2