Commit 82244f22 authored by sadrul@chromium.org's avatar sadrul@chromium.org

webkit: Check that delegate_ isn't NULL before trying to updateGeometry.

It appears that paint() checks for non-NULL delegate_, so it makes sense to
do the same for updateGeometry as well.

BUG=155950

Review URL: https://codereview.chromium.org/11148031

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@162262 0039d316-1c4b-4281-b951-d872f2087c98
parent 354a1428
...@@ -347,9 +347,9 @@ void WebPluginImpl::updateGeometry( ...@@ -347,9 +347,9 @@ void WebPluginImpl::updateGeometry(
} }
// Only UpdateGeometry if either the window or clip rects have changed. // Only UpdateGeometry if either the window or clip rects have changed.
if (first_geometry_update_ || if (delegate_ && (first_geometry_update_ ||
new_geometry.window_rect != geometry_.window_rect || new_geometry.window_rect != geometry_.window_rect ||
new_geometry.clip_rect != geometry_.clip_rect) { new_geometry.clip_rect != geometry_.clip_rect)) {
// Notify the plugin that its parameters have changed. // Notify the plugin that its parameters have changed.
delegate_->UpdateGeometry(new_geometry.window_rect, new_geometry.clip_rect); delegate_->UpdateGeometry(new_geometry.window_rect, new_geometry.clip_rect);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment