Remove notifications from MediaFileSystemRegistry.

Remove notifications from MediaFileSystemRegistry.

BUG=170921
TEST=everything still works

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=244301

[Lei Zhang, 2014-01-09 22:49:47]

lgtm

https://codereview.chromium.org/129743003/diff/40001/chrome/browser/media_gal...
File chrome/browser/media_galleries/media_file_system_registry.cc (right):

https://codereview.chromium.org/129743003/diff/40001/chrome/browser/media_gal...
chrome/browser/media_galleries/media_file_system_registry.cc:55: // interested
in. Once all of the RPHs have closed or been terminated a call
s/terminated/destroyed/

https://codereview.chromium.org/129743003/diff/40001/chrome/browser/media_gal...
chrome/browser/media_galleries/media_file_system_registry.cc:87: };
nit: blank line between class decl

https://codereview.chromium.org/129743003/diff/40001/chrome/browser/media_gal...
chrome/browser/media_galleries/media_file_system_registry.cc:95: size_t
ObserverCount() { return observed_web_contentses_.size(); }
We only need to know if |observed_web_contentses_| is empty or not.

https://codereview.chromium.org/129743003/diff/40001/chrome/browser/media_gal...
chrome/browser/media_galleries/media_file_system_registry.cc:177: if
(observed_web_contentses_.find(web_contents)
nit: ContainsKey()

[Avi (use Gerrit), 2014-01-10 15:42:23]

https://codereview.chromium.org/129743003/diff/40001/chrome/browser/media_gal...
File chrome/browser/media_galleries/media_file_system_registry.cc (right):

https://codereview.chromium.org/129743003/diff/40001/chrome/browser/media_gal...
chrome/browser/media_galleries/media_file_system_registry.cc:55: // interested
in. Once all of the RPHs have closed or been terminated a call
On 2014/01/09 22:49:48, Lei Zhang wrote:
> s/terminated/destroyed/

Done.

https://codereview.chromium.org/129743003/diff/40001/chrome/browser/media_gal...
chrome/browser/media_galleries/media_file_system_registry.cc:87: };
On 2014/01/09 22:49:48, Lei Zhang wrote:
> nit: blank line between class decl

Done.

https://codereview.chromium.org/129743003/diff/40001/chrome/browser/media_gal...
chrome/browser/media_galleries/media_file_system_registry.cc:95: size_t
ObserverCount() { return observed_web_contentses_.size(); }
On 2014/01/09 22:49:48, Lei Zhang wrote:
> We only need to know if |observed_web_contentses_| is empty or not.

Done.

[commit-bot: I haz the power, 2014-01-10 15:43:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/avi@chromium.org/129743003/190001

[commit-bot: I haz the power, 2014-01-10 16:21:49]

Retried try job too often on win_rel for step(s) app_list_unittests,
ash_unittests, aura_unittests, cacheinvalidation_unittests, cc_unittests,
check_deps, check_deps2git, chrome_elf_unittests, chromedriver_unittests,
components_unittests, compositor_unittests, content_browsertests,
content_unittests, crypto_unittests, events_unittests, google_apis_unittests,
gpu_unittests, installer_util_unittests, ipc_tests, jingle_unittests,
media_unittests, mini_installer_test, nacl_integration, ppapi_unittests,
printing_unittests, remoting_unittests, sql_unittests, sync_integration_tests,
sync_unit_tests, telemetry_unittests, views_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2014-01-10 16:55:38]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/avi@chromium.org/129743003/230002

[commit-bot: I haz the power, 2014-01-10 23:20:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/avi@chromium.org/129743003/230002

[commit-bot: I haz the power, 2014-01-11 01:58:47]

Change committed as 244301