diff options
Diffstat (limited to 'webkit/plugins/npapi')
-rw-r--r-- | webkit/plugins/npapi/webplugin_delegate_impl.cc | 10 | ||||
-rw-r--r-- | webkit/plugins/npapi/webplugin_impl.cc | 11 |
2 files changed, 15 insertions, 6 deletions
diff --git a/webkit/plugins/npapi/webplugin_delegate_impl.cc b/webkit/plugins/npapi/webplugin_delegate_impl.cc index 04f2193..c2c7697 100644 --- a/webkit/plugins/npapi/webplugin_delegate_impl.cc +++ b/webkit/plugins/npapi/webplugin_delegate_impl.cc @@ -1,4 +1,4 @@ -// Copyright (c) 2011 The Chromium Authors. All rights reserved. +// Copyright (c) 2012 The Chromium Authors. All rights reserved. // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. @@ -88,13 +88,17 @@ bool WebPluginDelegateImpl::Initialize( creation_succeeded_ = instance_->Start( url, argn.get(), argv.get(), argc, load_manually); - if (!creation_succeeded_) + if (!creation_succeeded_) { + VLOG(1) << "Couldn't start plug-in instance"; return false; + } windowless_ = instance_->windowless(); if (!windowless_) { - if (!WindowedCreatePlugin()) + if (!WindowedCreatePlugin()) { + VLOG(1) << "Couldn't create windowed plug-in"; return false; + } } else { // For windowless plugins we should set the containing window handle // as the instance window handle. This is what Safari does. Not having diff --git a/webkit/plugins/npapi/webplugin_impl.cc b/webkit/plugins/npapi/webplugin_impl.cc index 4049596..3fe5d1f 100644 --- a/webkit/plugins/npapi/webplugin_impl.cc +++ b/webkit/plugins/npapi/webplugin_impl.cc @@ -1,4 +1,4 @@ -// Copyright (c) 2011 The Chromium Authors. All rights reserved. +// Copyright (c) 2012 The Chromium Authors. All rights reserved. // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. @@ -247,13 +247,17 @@ struct WebPluginImpl::ClientInfo { }; bool WebPluginImpl::initialize(WebPluginContainer* container) { - if (!page_delegate_) + if (!page_delegate_) { + LOG(ERROR) << "No page delegate"; return false; + } WebPluginDelegate* plugin_delegate = page_delegate_->CreatePluginDelegate( file_path_, mime_type_); - if (!plugin_delegate) + if (!plugin_delegate) { + LOG(ERROR) << "Couldn't create plug-in delegate"; return false; + } // Set the container before Initialize because the plugin may // synchronously call NPN_GetValue to get its container during its @@ -262,6 +266,7 @@ bool WebPluginImpl::initialize(WebPluginContainer* container) { bool ok = plugin_delegate->Initialize( plugin_url_, arg_names_, arg_values_, this, load_manually_); if (!ok) { + LOG(ERROR) << "Couldn't initialize plug-in"; plugin_delegate->PluginDestroyed(); return false; } |