Revert of Refine ash shelf tooltip closing on non-mash ChromeOS.

Revert of Refine ash shelf tooltip closing on non-mash ChromeOS. (patchset #5 id:80001 of https://codereview.chromium.org/1828133004/ )

Reason for revert:
ash_unittests ShelfLayoutManagerTest.ShutdownHandlesWindowActivation failure on win:
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/47284

Original issue's description:
> Refine ash shelf tooltip closing on non-mash ChromeOS.
>
> Fixes a regression from: https://codereview.chromium.org/1816753002
> Also reverse expected behavior for touching tooltips.
> (closes on stable, tests expected them to stay open...)
>
> Inline bubble function definitions.
> Call WillDeleteShelf earlier to fix a test crash...
> TODO: Encapsulate more ShelfLayoutManager code.
>
> BUG=595853
> TEST=ChromeOS tooltips stay open on hover; close for external touches.
> R=sky@chromium.org
>
> Committed: https://crrev.com/1a0d283fa81e19b47fb85523f28c4a03f7dc1d24
> Cr-Commit-Position: refs/heads/master@{#383404}
>
> Committed: https://crrev.com/c2aa7f5e8870c04c8aff0fd5da27820636e53e38
> Cr-Commit-Position: refs/heads/master@{#383579}

TBR=sky@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=595853
Committed: https://crrev.com/327ea5e179d66d730bd6e03486b3494660d84dbc
Cr-Commit-Position: refs/heads/master@{#383622}

[msw, 2016-03-29 00:04:05]

Created Revert of Refine ash shelf tooltip closing on non-mash ChromeOS.

[commit-bot: I haz the power, 2016-03-29 00:04:26]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1834203003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1834203003/1

[commit-bot: I haz the power, 2016-03-29 00:04:53]

Description was changed from

==========
Revert of Refine ash shelf tooltip closing on non-mash ChromeOS. (patchset #5
id:80001 of https://codereview.chromium.org/1828133004/ )

Reason for revert:
ash_unittests ShelfLayoutManagerTest.ShutdownHandlesWindowActivation failure on
win:
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...

Original issue's description:
> Refine ash shelf tooltip closing on non-mash ChromeOS.
> 
> Fixes a regression from: https://codereview.chromium.org/1816753002
> Also reverse expected behavior for touching tooltips.
> (closes on stable, tests expected them to stay open...)
> 
> Inline bubble function definitions.
> Call WillDeleteShelf earlier to fix a test crash...
> TODO: Encapsulate more ShelfLayoutManager code.
> 
> BUG=595853
> TEST=ChromeOS tooltips stay open on hover; close for external touches.
> R=sky@chromium.org
> 
> Committed: https://crrev.com/1a0d283fa81e19b47fb85523f28c4a03f7dc1d24
> Cr-Commit-Position: refs/heads/master@{#383404}
> 
> Committed: https://crrev.com/c2aa7f5e8870c04c8aff0fd5da27820636e53e38
> Cr-Commit-Position: refs/heads/master@{#383579}

TBR=sky@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=595853
==========

to

==========
Revert of Refine ash shelf tooltip closing on non-mash ChromeOS. (patchset #5
id:80001 of https://codereview.chromium.org/1828133004/ )

Reason for revert:
ash_unittests ShelfLayoutManagerTest.ShutdownHandlesWindowActivation failure on
win:
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...

Original issue's description:
> Refine ash shelf tooltip closing on non-mash ChromeOS.
> 
> Fixes a regression from: https://codereview.chromium.org/1816753002
> Also reverse expected behavior for touching tooltips.
> (closes on stable, tests expected them to stay open...)
> 
> Inline bubble function definitions.
> Call WillDeleteShelf earlier to fix a test crash...
> TODO: Encapsulate more ShelfLayoutManager code.
> 
> BUG=595853
> TEST=ChromeOS tooltips stay open on hover; close for external touches.
> R=sky@chromium.org
> 
> Committed: https://crrev.com/1a0d283fa81e19b47fb85523f28c4a03f7dc1d24
> Cr-Commit-Position: refs/heads/master@{#383404}
> 
> Committed: https://crrev.com/c2aa7f5e8870c04c8aff0fd5da27820636e53e38
> Cr-Commit-Position: refs/heads/master@{#383579}

TBR=sky@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=595853
==========

[commit-bot: I haz the power, 2016-03-29 00:04:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-29 00:07:40]

Description was changed from

==========
Revert of Refine ash shelf tooltip closing on non-mash ChromeOS. (patchset #5
id:80001 of https://codereview.chromium.org/1828133004/ )

Reason for revert:
ash_unittests ShelfLayoutManagerTest.ShutdownHandlesWindowActivation failure on
win:
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...

Original issue's description:
> Refine ash shelf tooltip closing on non-mash ChromeOS.
> 
> Fixes a regression from: https://codereview.chromium.org/1816753002
> Also reverse expected behavior for touching tooltips.
> (closes on stable, tests expected them to stay open...)
> 
> Inline bubble function definitions.
> Call WillDeleteShelf earlier to fix a test crash...
> TODO: Encapsulate more ShelfLayoutManager code.
> 
> BUG=595853
> TEST=ChromeOS tooltips stay open on hover; close for external touches.
> R=sky@chromium.org
> 
> Committed: https://crrev.com/1a0d283fa81e19b47fb85523f28c4a03f7dc1d24
> Cr-Commit-Position: refs/heads/master@{#383404}
> 
> Committed: https://crrev.com/c2aa7f5e8870c04c8aff0fd5da27820636e53e38
> Cr-Commit-Position: refs/heads/master@{#383579}

TBR=sky@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=595853
==========

to

==========
Revert of Refine ash shelf tooltip closing on non-mash ChromeOS. (patchset #5
id:80001 of https://codereview.chromium.org/1828133004/ )

Reason for revert:
ash_unittests ShelfLayoutManagerTest.ShutdownHandlesWindowActivation failure on
win:
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...

Original issue's description:
> Refine ash shelf tooltip closing on non-mash ChromeOS.
>
> Fixes a regression from: https://codereview.chromium.org/1816753002
> Also reverse expected behavior for touching tooltips.
> (closes on stable, tests expected them to stay open...)
>
> Inline bubble function definitions.
> Call WillDeleteShelf earlier to fix a test crash...
> TODO: Encapsulate more ShelfLayoutManager code.
>
> BUG=595853
> TEST=ChromeOS tooltips stay open on hover; close for external touches.
> R=sky@chromium.org
>
> Committed: https://crrev.com/1a0d283fa81e19b47fb85523f28c4a03f7dc1d24
> Cr-Commit-Position: refs/heads/master@{#383404}
>
> Committed: https://crrev.com/c2aa7f5e8870c04c8aff0fd5da27820636e53e38
> Cr-Commit-Position: refs/heads/master@{#383579}

TBR=sky@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=595853

Committed: https://crrev.com/327ea5e179d66d730bd6e03486b3494660d84dbc
Cr-Commit-Position: refs/heads/master@{#383622}
==========

[commit-bot: I haz the power, 2016-03-29 00:07:41]

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