Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(575)

Unified Diff: content/browser/renderer_host/render_widget_host_view_aura.cc

Issue 12612005: Revert 186627 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 7 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/render_widget_host_view_aura.cc
===================================================================
--- content/browser/renderer_host/render_widget_host_view_aura.cc (revision 186756)
+++ content/browser/renderer_host/render_widget_host_view_aura.cc (working copy)
@@ -289,7 +289,10 @@
ack.gl_frame_data.reset(new cc::GLFrameData());
DCHECK(!texture_to_produce || !skip_frame);
if (texture_to_produce) {
- ack.gl_frame_data->mailbox = texture_to_produce->Produce();
+ std::string mailbox_name = texture_to_produce->Produce();
+ std::copy(mailbox_name.data(),
+ mailbox_name.data() + mailbox_name.length(),
+ reinterpret_cast<char*>(ack.gl_frame_data->mailbox.name));
ack.gl_frame_data->size = texture_to_produce->size();
ack.gl_frame_data->sync_point =
content::ImageTransportFactory::GetInstance()->InsertSyncPoint();
@@ -307,7 +310,7 @@
void AcknowledgeBufferForGpu(
int32 route_id,
int gpu_host_id,
- const gpu::Mailbox& received_mailbox,
+ const std::string& received_mailbox,
bool skip_frame,
const scoped_refptr<ui::Texture>& texture_to_produce) {
AcceleratedSurfaceMsg_BufferPresented_Params ack;
@@ -1161,7 +1164,7 @@
bool RenderWidgetHostViewAura::SwapBuffersPrepare(
const gfx::Rect& surface_rect,
const gfx::Rect& damage_rect,
- const gpu::Mailbox& mailbox_name,
+ const std::string& mailbox_name,
const BufferPresentedCallback& ack_callback) {
if (last_swapped_surface_size_ != surface_rect.size()) {
// The surface could have shrunk since we skipped an update, in which
@@ -1172,7 +1175,7 @@
}
if (ShouldSkipFrame(ConvertSizeToDIP(this, surface_rect.size())) ||
- mailbox_name.IsZero()) {
+ mailbox_name.empty()) {
skipped_damage_.op(RectToSkIRect(damage_rect), SkRegion::kUnion_Op);
ack_callback.Run(true, scoped_refptr<ui::Texture>());
return false;
@@ -1314,13 +1317,16 @@
ImageTransportFactory* factory = ImageTransportFactory::GetInstance();
factory->WaitSyncPoint(frame->gl_frame_data->sync_point);
+ std::string mailbox_name(
+ reinterpret_cast<const char*>(frame->gl_frame_data->mailbox.name),
+ sizeof(frame->gl_frame_data->mailbox.name));
BuffersSwapped(
- frame->gl_frame_data->size, frame->gl_frame_data->mailbox, ack_callback);
+ frame->gl_frame_data->size, mailbox_name, ack_callback);
}
void RenderWidgetHostViewAura::BuffersSwapped(
const gfx::Size& size,
- const gpu::Mailbox& mailbox_name,
+ const std::string& mailbox_name,
const BufferPresentedCallback& ack_callback) {
scoped_refptr<ui::Texture> texture_to_return(current_surface_);
const gfx::Rect surface_rect = gfx::Rect(gfx::Point(), size);
« no previous file with comments | « content/browser/renderer_host/render_widget_host_view_aura.h ('k') | content/common/browser_plugin/browser_plugin_messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698