diff options
| author | bbudge@chromium.org <bbudge@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-11-02 21:41:20 +0000 |
|---|---|---|
| committer | bbudge@chromium.org <bbudge@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-11-02 21:41:20 +0000 |
| commit | c0514eec5ab9c7eef742cadf0d94671e192b8301 (patch) | |
| tree | e8e1c1f5d7b5ae19f0e5bba8b05e543e8fd24682 /ppapi | |
| parent | edcfae50db024f073d6d140a060dba82ee7f2b10 (diff) | |
| download | chromium_src-c0514eec5ab9c7eef742cadf0d94671e192b8301.zip chromium_src-c0514eec5ab9c7eef742cadf0d94671e192b8301.tar.gz chromium_src-c0514eec5ab9c7eef742cadf0d94671e192b8301.tar.bz2 | |
DLOG rather than DCHECK in GetBrowserInterface when name is NULL.
Tests exercise this code path.
BUG=116317
TEST=nacl_integration
Review URL: https://chromiumcodereview.appspot.com/11366065
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@165756 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi')
| -rw-r--r-- | ppapi/proxy/plugin_dispatcher.cc | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/ppapi/proxy/plugin_dispatcher.cc b/ppapi/proxy/plugin_dispatcher.cc index 908e1d6..1c1eedc 100644 --- a/ppapi/proxy/plugin_dispatcher.cc +++ b/ppapi/proxy/plugin_dispatcher.cc @@ -105,9 +105,12 @@ PluginDispatcher* PluginDispatcher::GetForResource(const Resource* resource) { // static const void* PluginDispatcher::GetBrowserInterface(const char* interface_name) { - DCHECK(interface_name) << "|interface_name| is null. Did you forget to add " - "the |interface_name()| template function to the interface's C++ " - "wrapper?"; + if (!interface_name) { + DLOG(WARNING) << "|interface_name| is null. Did you forget to add " + "the |interface_name()| template function to the interface's C++ " + "wrapper?"; + return NULL; + } return InterfaceList::GetInstance()->GetInterfaceForPPB(interface_name); } |