For Mac, add AppListViewDelegateObserver::OnShutdown

For Mac, add AppListViewDelegateObserver::OnShutdown; deflake tests

We used to just leak everything on Mac. Since r295194, we clean up
references to Profiles being deleted, in Chrome's AppListViewDelegate.
This has caught out the Mac app list trying to access stuff it shouldn't
if Cocoa is still drawing it while Chrome is shutting down. Tests
started failing flakily.

This CL adds AppListViewDelegateObserver::OnShutdown to that Chrome's
AppListViewDelegate can notify the AppListView that it's shutting down.

On Views platforms, the Widget is just closed (which also happens just
after the chrome::NOTIFICATION_APP_TERMINATING broadcast). However, (a)
Cocoa doesn't have a `CloseAllSecondaryWidgets` call to trigger this and
(b) just closing the NSWindow isn't usually enough on Cocoa to avoid
accessing C++ objects due to reference counting.

We could also go through the platform-specific AppListService (e.g. by
passing in the AppListService that creates a Chrome
AppListViewDelegate), but this seemed neater.

This de-flakes a bunch of applist tests on Mac. The Mac-specific
ShowAppListUsingShim test is also augmented with extra checks for the
OnShutdown stuff.

BUG=415264
TEST=AppListServiceMacInteractiveTest.ShowAppListUsingShim,
AppListServiceInteractiveTest.*

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

[tapted, 2014-09-23 06:38:09]

Patchset #2 (id:20001) has been deleted

[tapted, 2014-09-23 06:38:29]

Patchset #2 (id:40001) has been deleted

[tapted, 2014-09-24 06:55:31]

tapted@chromium.org changed reviewers:
+ jackhou@chromium.org

[jackhou1, 2014-09-24 07:11:40]

lgtm

[tapted, 2014-09-24 07:15:13]

The CQ bit was checked by tapted@chromium.org

[commit-bot: I haz the power, 2014-09-24 07:15:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/593563002/140001

[commit-bot: I haz the power, 2014-09-24 07:40:33]

Committed patchset #6 (id:140001) as e208b90a52559e73ef05a322f5ee16221aca0bdb

[commit-bot: I haz the power, 2014-09-24 07:41:24]

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