diff options
| author | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-06-15 00:31:26 +0000 |
|---|---|---|
| committer | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-06-15 00:31:26 +0000 |
| commit | 0478f921566816ecb5a1974f3eee74a80326b860 (patch) | |
| tree | 79576ed0b6995b781c6465d590faac875f7e46ff /content | |
| parent | 737aadee4ad7d9c4e47c8d6d0c8a57973e974ab2 (diff) | |
| download | chromium_src-0478f921566816ecb5a1974f3eee74a80326b860.zip chromium_src-0478f921566816ecb5a1974f3eee74a80326b860.tar.gz chromium_src-0478f921566816ecb5a1974f3eee74a80326b860.tar.bz2 | |
Renames NOTIFICATION_RENDER_WIDGET_HOST_DID_PAINT to
NOTIFICATION_RENDER_WIDGET_HOST_DID_UPDATE_BACKING_STORE and makes it
be sent even if the view is hidden. This resulted in tweaking one
caller since it only wants to do something when the view is
showing. For the other places it doesn't seem to matter.
BUG=none
TEST=none
R=jam@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10536183
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142287 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content')
| -rw-r--r-- | content/browser/renderer_host/render_widget_host_impl.cc | 10 | ||||
| -rw-r--r-- | content/public/browser/notification_types.h | 6 |
2 files changed, 8 insertions, 8 deletions
diff --git a/content/browser/renderer_host/render_widget_host_impl.cc b/content/browser/renderer_host/render_widget_host_impl.cc index 668b75b..5cc4cf5 100644 --- a/content/browser/renderer_host/render_widget_host_impl.cc +++ b/content/browser/renderer_host/render_widget_host_impl.cc @@ -1355,6 +1355,11 @@ void RenderWidgetHostImpl::DidUpdateBackingStore( if (view_) view_->MovePluginWindows(params.plugin_window_moves); + NotificationService::current()->Notify( + NOTIFICATION_RENDER_WIDGET_HOST_DID_UPDATE_BACKING_STORE, + Source<RenderWidgetHost>(this), + NotificationService::NoDetails()); + // We don't need to update the view if the view is hidden. We must do this // early return after the ACK is sent, however, or the renderer will not send // us more data. @@ -1369,11 +1374,6 @@ void RenderWidgetHostImpl::DidUpdateBackingStore( view_being_painted_ = false; } - NotificationService::current()->Notify( - NOTIFICATION_RENDER_WIDGET_HOST_DID_PAINT, - Source<RenderWidgetHost>(this), - NotificationService::NoDetails()); - // If we got a resize ack, then perhaps we have another resize to send? bool is_resize_ack = ViewHostMsg_UpdateRect_Flags::is_resize_ack(params.flags); diff --git a/content/public/browser/notification_types.h b/content/public/browser/notification_types.h index 6d9c080..9c4a835 100644 --- a/content/public/browser/notification_types.h +++ b/content/public/browser/notification_types.h @@ -276,9 +276,9 @@ enum NotificationType { // the RenderWidgetHost, the details are not used. NOTIFICATION_RENDER_WIDGET_HOST_DESTROYED, - // Sent after the widget has painted. The source is the RenderWidgetHost, - // the details are not used. - NOTIFICATION_RENDER_WIDGET_HOST_DID_PAINT, + // Sent after the backing store has been updated but before the widget has + // painted. The source is the RenderWidgetHost, the details are not used. + NOTIFICATION_RENDER_WIDGET_HOST_DID_UPDATE_BACKING_STORE, // This notifies the observer that a PaintAtSizeACK was received. The source // is the RenderWidgetHost, the details are an instance of |