Revert of Refactor DoesIntersectRect into BrowserNonClientFrameView.

Revert of Refactor DoesIntersectRect into BrowserNonClientFrameView. (patchset #2 id:20001 of https://codereview.chromium.org/2262563002/ )

Reason for revert:
It looks like I broke V1 apps on Chrome OS (see crbug.com/640230) which means it's probably broken for Mus as well (though it's hard to tell) and that negates the benefit of this refactor. I'm very sad.

Original issue's description:
> Refactor DoesIntersectRect into BrowserNonClientFrameView.
>
> As far as I can tell the DoesIntersectRect implementations of
> OpaqueBrowserFrameView, BrowserNonClientFrameViewMus, and
> BrowserNonClientFrameViewAsh are all behaviorly identical but entirely
> duplicated. So I moved them all into their common base class. Nothing
> should look or act differently after this change.
>
> TEST=Check tab strip, toolbar, and window controls to make sure still work correctly.
>
> Committed: https://crrev.com/55e6cc560a8e152d26f1b69e0929c53681eeacd7
> Cr-Commit-Position: refs/heads/master@{#413514}

TBR=sky@chromium.org,msw@chromium.org,varkha@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.

Committed: https://crrev.com/d449eed616d0d991d72cfb3faf23763a18a4b5dd
Cr-Commit-Position: refs/heads/master@{#413898}

[Bret, 2016-08-23 21:40:05]

The CQ bit was checked by bsep@chromium.org

[Bret, 2016-08-23 21:40:06]

Created Revert of Refactor DoesIntersectRect into BrowserNonClientFrameView.

[commit-bot: I haz the power, 2016-08-23 21:40:39]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[varkha, 2016-08-23 21:49:17]

lgtm (but I am not an owner).

[msw, 2016-08-23 21:50:15]

lgtm; hopefully you can reland with a fix for that issue.

[commit-bot: I haz the power, 2016-08-23 23:37:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-23 23:42:24]

Description was changed from

==========
Revert of Refactor DoesIntersectRect into BrowserNonClientFrameView. (patchset
#2 id:20001 of https://codereview.chromium.org/2262563002/ )

Reason for revert:
It looks like I broke V1 apps on Chrome OS (see crbug.com/640230) which means
it's probably broken for Mus as well (though it's hard to tell) and that negates
the benefit of this refactor. I'm very sad.

Original issue's description:
> Refactor DoesIntersectRect into BrowserNonClientFrameView.
> 
> As far as I can tell the DoesIntersectRect implementations of
> OpaqueBrowserFrameView, BrowserNonClientFrameViewMus, and
> BrowserNonClientFrameViewAsh are all behaviorly identical but entirely
> duplicated. So I moved them all into their common base class. Nothing
> should look or act differently after this change.
> 
> TEST=Check tab strip, toolbar, and window controls to make sure still work
correctly.
> 
> Committed: https://crrev.com/55e6cc560a8e152d26f1b69e0929c53681eeacd7
> Cr-Commit-Position: refs/heads/master@{#413514}

TBR=sky@chromium.org,msw@chromium.org,varkha@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

to

==========
Revert of Refactor DoesIntersectRect into BrowserNonClientFrameView. (patchset
#2 id:20001 of https://codereview.chromium.org/2262563002/ )

Reason for revert:
It looks like I broke V1 apps on Chrome OS (see crbug.com/640230) which means
it's probably broken for Mus as well (though it's hard to tell) and that negates
the benefit of this refactor. I'm very sad.

Original issue's description:
> Refactor DoesIntersectRect into BrowserNonClientFrameView.
>
> As far as I can tell the DoesIntersectRect implementations of
> OpaqueBrowserFrameView, BrowserNonClientFrameViewMus, and
> BrowserNonClientFrameViewAsh are all behaviorly identical but entirely
> duplicated. So I moved them all into their common base class. Nothing
> should look or act differently after this change.
>
> TEST=Check tab strip, toolbar, and window controls to make sure still work
correctly.
>
> Committed: https://crrev.com/55e6cc560a8e152d26f1b69e0929c53681eeacd7
> Cr-Commit-Position: refs/heads/master@{#413514}

TBR=sky@chromium.org,msw@chromium.org,varkha@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.

Committed: https://crrev.com/d449eed616d0d991d72cfb3faf23763a18a4b5dd
Cr-Commit-Position: refs/heads/master@{#413898}
==========

[commit-bot: I haz the power, 2016-08-23 23:42:26]

Patchset 1 (id:??) landed as
https://crrev.com/d449eed616d0d991d72cfb3faf23763a18a4b5dd
Cr-Commit-Position: refs/heads/master@{#413898}