Modifying ArgumentNullOrEmpty message to be consistent with Mvc 5 \
testing code.
Don't lock metadata references
Change `ViewDataDictionary` copy constructor to ensure `ModelMetadata` is
 never that for `object`
- `ViewDataDictionary<TModel>.ModelMetadata` was for `object` after base
 copy constructor got value from `ViewDataDictionary<object>`
- problem led to #1426 symptoms
- with copy constructor leaving `base.ModelMetadata==null` more often,
 `ViewDataDictionary<TModel>.ModelMetadata` usually tracks `TModel` if
 `Model==null`

nit:
- fix existing comment in main `ViewDataDictionary` copy constructor
Add "/home/nulluser" view to MVC sample
- exercises display and editor helpers when `Model==null`
Add more unit tests of `ViewDataDictionary` copy constructors
Merge branch 'release' into dev
* Porting Hidden and HiddenFor unit tests

Partial fix to #453
Reacting to Roslyn version changes
Merge branch 'release' into dev
Change logging in Mvc to log with TraceType.Verbose
Fixes #1457
Merge branch 'release' into dev
Fix for #1448 - ViewComponents must be public top-level classes

ViewComponents and Controllers now follow the same rules exactly for what
types of classes they can be.

Also corrected a bug in a test for controllers. Closed-generic types can
be controllers, the test was wrong.
TODO removal part 1 - ActionExecutorTests
Fix for #1442 - Find better names for ApiExplorerProperties

Taking the suggestion here to move these to a sub-object. This is future
proof in the event that we need to capture more data for ApiExplorer, and
reads better.
Added a functional test for view precompilation
Revert "Added a functional test for view precompilation"

This reverts commit 310feeb826b3a5ea489925ca83628e96f8af5817.
Update to JSON.Net 6.0.6

Fixes #1473
Merge branch 'release' into dev
Added description in project.json for all projects
Merge branch 'release' into dev