Remove DCHECK in DialRegistry dtor. This may trigger on browser shutdown if

Remove DCHECK in DialRegistry dtor.  This may trigger on browser shutdown if
there are event listeners still registered with chrome.dial.  This is not an
issue as all owned objects will be cleaned up anyway by following the normal dtor
sequence DialAPI -> DialRegistry -> DialService.

BUG=460197
Committed: https://crrev.com/06dc4c4f1e654a1a3a4a30da8ecfe016beb9413b
Cr-Commit-Position: refs/heads/master@{#317194}

[mark a. foltz, 2015-02-19 20:11:52]

mfoltz@chromium.org changed reviewers:
+ imcheng@google.com, wez@chromium.org

[Wez, 2015-02-19 22:48:32]

lgtm

https://codereview.chromium.org/916103007/diff/20001/chrome/browser/extension...
File chrome/browser/extensions/api/dial/dial_api.h (right):

https://codereview.chromium.org/916103007/diff/20001/chrome/browser/extension...
chrome/browser/extensions/api/dial/dial_api.h:27: // managing here;
ShutdownOnUIThread below is a no-op.
This comment's pretty confusing; are you saying that there are no resources that
this class manages that require UI thread teardown? Or something else?

[mark a. foltz, 2015-02-19 23:04:29]

On 2015/02/19 22:48:32, Wez wrote:
> lgtm
> 
>
https://codereview.chromium.org/916103007/diff/20001/chrome/browser/extension...
> File chrome/browser/extensions/api/dial/dial_api.h (right):
> 
>
https://codereview.chromium.org/916103007/diff/20001/chrome/browser/extension...
> chrome/browser/extensions/api/dial/dial_api.h:27: // managing here;
> ShutdownOnUIThread below is a no-op.
> This comment's pretty confusing; are you saying that there are no resources
that
> this class manages that require UI thread teardown? Or something else?

Simplified comment.  Basically this object is not holding onto any references
that need to be dropped during browser shutdown.

[mark a. foltz, 2015-02-19 23:05:00]

New patchsets have been uploaded after l-g-t-m from wez@chromium.org

[mark a. foltz, 2015-02-19 23:17:33]

The CQ bit was checked by mfoltz@chromium.org

[commit-bot: I haz the power, 2015-02-19 23:18:30]

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

[commit-bot: I haz the power, 2015-02-20 00:41:14]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-02-20 00:41:42]

Patchset 3 (id:??) landed as
https://crrev.com/06dc4c4f1e654a1a3a4a30da8ecfe016beb9413b
Cr-Commit-Position: refs/heads/master@{#317194}