Temporary fix for bad hit testing of tab close button

The problem is the close button ends up saying it has a hit test mask,
but returns an empty hit test mask. The code was checking for
containment, when it should be checking for intersection.

My fix isn't ideal, we really need to return the intersection, but
this should be good for now.

BUG=332334
TEST=see bug
R=msw@chromium.org, sadrul@chromium.org, tdanderson@chromium.org

Review URL: https://codereview.chromium.org/134203004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@246686 0039d316-1c4b-4281-b951-d872f2087c98
Docserver: Convert DocumentRendererTest to using a full ServerInstance in its
SetUp rather than a fake class. This is both (slightly) more meaningful, and
as DocumentRenderer is about to get more complex, better for then.
Also fix a malformed HTML tag causing presubmit checks to fail.

R=yoz@chromium.org
NOTRY=true

Review URL: https://codereview.chromium.org/139613004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@246687 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 246686
SVN changes up to revision 246687