Removing unnecessary character striping from transaction name

Now that we're fully JSON encoding the payload, don't need to mess with it directly
Shouldn't have removed those to_f's...
RUBY-874 Kill all whitespace
Adjust initial spacing on lines

Doc site post we make from CI doesn't render right with the bullets preceded by two spaces.
Trim those off so things are nice and clean straight through.
ACDC-7 Add a little more detail to comment in license_test
ACDC-7 One more license check
RUBY-1050 Avoid corrupting the scope stack when HTTP requests fail

When traced a trace HTTP request fails due to a network error, timeout,
or similar, we need to ensure that we still pop the scope stack so
that we can move on with tracing of the transaction. The failure may
be rescued by user code at a higher level, so the transaction might
continue without an error propagating.

Conflicts:
	lib/new_relic/agent/cross_app_tracing.rb
RUBY-1050 Conditionalize cross-app metric recording on having a response to avoid scary log msg

Conflicts:
	lib/new_relic/agent/cross_app_tracing.rb
Tag seen exceptions with an ivar rather than checking object_ids

We tag exception objects to avoid duplicates, but there are a few
constraints:

1. We don't want to actually hold a reference to the exception objects
   to avoid unnecessarily retaining them.
2. We don't want the tagging to result in false positives (that is, we
   shouldn't incorrectly flag an exception as a duplicate if it isn't)
3. We want the tagging itself to be fast.

The previous approach satisfied constraints 1 and 3, but not 2, since
we were maintaining a list of seen object_ids (cleared periodically),
and object_ids can be reused in Ruby once the objects they're
associated with are GC'd. The ivar approach satisfies all three
constraints.