Do not allow creating dummy notifier id from production, for safety.

Do not allow creating dummy notifier id in production.

Somehow NotifierId() default constructor still exists but this is
really misleading, because it creates an id only for test use.
To apply this rule explicitly, move this constructor to private.

BUG=327033
R=dewittj@chromium.org
TBR=jamescook@chromium.org
TEST=no functional change, compile succeeds

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

[dewittj, 2013-12-10 17:03:15]

lgtm

[Jun Mukai, 2013-12-10 18:24:14]

+jamescook as TBR, the change of ash/shell/window_type_launcher.cc is trivial.
cc: rsesek for the change of tests in ui/message_center/cocoa, changes for
namespaces.

[commit-bot: I haz the power, 2013-12-10 18:25:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mukai@chromium.org/101473003/100001

[James Cook, 2013-12-10 18:43:12]

lgtm

[commit-bot: I haz the power, 2013-12-10 19:51:57]

Retried try job too often on linux_chromeos for step(s) app_list_unittests,
ash_unittests, aura_unittests, base_unittests, browser_tests,
cacheinvalidation_unittests, check_deps, chromeos_unittests,
components_unittests, content_browsertests, content_unittests, crypto_unittests,
dbus_unittests, device_unittests, events_unittests, google_apis_unittests,
gpu_unittests, interactive_ui_tests, ipc_tests, jingle_unittests,
media_unittests, net_unittests, ppapi_unittests, printing_unittests,
sandbox_linux_unittests, sql_unittests, sync_unit_tests, unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[commit-bot: I haz the power, 2013-12-10 21:44:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mukai@chromium.org/101473003/120001

[commit-bot: I haz the power, 2013-12-11 03:22:23]

Change committed as 239993