Destroy MediaFileSystemRegistry before StorageMonitor in MTPDeviceDelegateImplWinTest.

Destroy MediaFileSystemRegistry before StorageMonitor in MTPDeviceDelegateImplWinTest.

MediaFileSystemRegistry must be destroyed before StorageMonitor
because it calls StorageMonitor::RemoveInstance in its destructor.

It's not clear why this test isn't already failing on the waterfall.
A developer hit a failure when adding unrelated tests (see bug).

BUG=686803
Review-Url: https://codereview.chromium.org/2664073003
Cr-Commit-Position: refs/heads/master@{#447319}
Committed: https://chromium.googlesource.com/chromium/src/+/4efece94336e6bd0dccfc2772a6b0ab856aef84c

[fdoray, 2017-01-31 16:23:28]

Description was changed from

==========
Destroy MediaFileSystemRegistry before StorageMonitor in
MTPDeviceDelegateImplWinTest.

BUG=
==========

to

==========
Destroy MediaFileSystemRegistry before StorageMonitor in
MTPDeviceDelegateImplWinTest.

MediaFileSystemRegistry must be destroyed before StorageMonitor
because it calls StorageMonitor::RemoveInstance in its destructor.


BUG=686803
==========

[fdoray, 2017-01-31 16:23:33]

The CQ bit was checked by fdoray@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-31 16:23:44]

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

[commit-bot: I haz the power, 2017-01-31 16:59:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-31 16:59:43]

Dry run: This issue passed the CQ dry run.

[fdoray, 2017-01-31 18:47:30]

Description was changed from

==========
Destroy MediaFileSystemRegistry before StorageMonitor in
MTPDeviceDelegateImplWinTest.

MediaFileSystemRegistry must be destroyed before StorageMonitor
because it calls StorageMonitor::RemoveInstance in its destructor.


BUG=686803
==========

to

==========
Destroy MediaFileSystemRegistry before StorageMonitor in
MTPDeviceDelegateImplWinTest.

MediaFileSystemRegistry must be destroyed before StorageMonitor
because it calls StorageMonitor::RemoveInstance in its destructor.

It's not clear why this test isn't already failing on the waterfall.
A developer hit a failure when adding unrelated tests (see bug).

BUG=686803
==========

[fdoray, 2017-01-31 18:47:44]

fdoray@chromium.org changed reviewers:
+ reillyg@chromium.org

[fdoray, 2017-01-31 18:47:45]

PTAL

[Reilly Grant (use Gerrit), 2017-01-31 20:36:14]

lgtm

[fdoray, 2017-01-31 20:36:57]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2017-01-31 20:37:39]

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

[commit-bot: I haz the power, 2017-01-31 20:42:49]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485895017755010, "parent_rev":
"bc979776bf3949f8111d18f3e476833de024d8e1", "commit_rev":
"4efece94336e6bd0dccfc2772a6b0ab856aef84c"}

[commit-bot: I haz the power, 2017-01-31 20:43:25]

Description was changed from

==========
Destroy MediaFileSystemRegistry before StorageMonitor in
MTPDeviceDelegateImplWinTest.

MediaFileSystemRegistry must be destroyed before StorageMonitor
because it calls StorageMonitor::RemoveInstance in its destructor.

It's not clear why this test isn't already failing on the waterfall.
A developer hit a failure when adding unrelated tests (see bug).

BUG=686803
==========

to

==========
Destroy MediaFileSystemRegistry before StorageMonitor in
MTPDeviceDelegateImplWinTest.

MediaFileSystemRegistry must be destroyed before StorageMonitor
because it calls StorageMonitor::RemoveInstance in its destructor.

It's not clear why this test isn't already failing on the waterfall.
A developer hit a failure when adding unrelated tests (see bug).

BUG=686803

Review-Url: https://codereview.chromium.org/2664073003
Cr-Commit-Position: refs/heads/master@{#447319}
Committed:
https://chromium.googlesource.com/chromium/src/+/4efece94336e6bd0dccfc2772a6b...
==========

[commit-bot: I haz the power, 2017-01-31 20:43:26]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/4efece94336e6bd0dccfc2772a6b...

[tsergeant, 2017-01-31 22:54:58]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2662343003/ by tsergeant@chromium.org.

The reason for reverting is: This CL is causing the test
MTPDeviceDelegateImplWinTest.GalleryNameMTP to fail on Windows 7. See:

https://luci-milo.appspot.com/buildbot/chromium.win/Win7%20Tests%20%281%29/63111.