diff options
| author | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-07-01 16:41:11 +0000 |
|---|---|---|
| committer | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-07-01 16:41:11 +0000 |
| commit | f24c3538ca22551b1509895903c70a011731467e (patch) | |
| tree | 6d1d9760d10adc931f4bc2a5ce973d75a366fbb8 | |
| parent | 0c98c465235af557199b1dc35d864a1f8996fba4 (diff) | |
| download | chromium_src-f24c3538ca22551b1509895903c70a011731467e.zip chromium_src-f24c3538ca22551b1509895903c70a011731467e.tar.gz chromium_src-f24c3538ca22551b1509895903c70a011731467e.tar.bz2 | |
Coverity: Fix a REVERSE_INULL.
CID=17773
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/7290028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91306 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/renderer/extensions/event_bindings.cc | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/chrome/renderer/extensions/event_bindings.cc b/chrome/renderer/extensions/event_bindings.cc index 911a3af..0d2c344 100644 --- a/chrome/renderer/extensions/event_bindings.cc +++ b/chrome/renderer/extensions/event_bindings.cc @@ -199,12 +199,12 @@ static bool HasSufficientPermissions(ContextInfo* context, return true; RenderView* renderview = bindings_utils::GetRenderViewForCurrentContext(); + if (!renderview) + return false; + WebDocument document = renderview->webview()->mainFrame()->document(); - bool url_permissions_ok = (!event_url.is_valid() || - (renderview && - GURL(document.url()).SchemeIs(chrome::kExtensionScheme) && - document.securityOrigin().canRequest(event_url))); - return url_permissions_ok; + return GURL(document.url()).SchemeIs(chrome::kExtensionScheme) && + document.securityOrigin().canRequest(event_url); } } // namespace |