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

Unified Diff: remoting/client/plugin/pepper_view.cc

Issue 3020038: Revert 53892 - Initial scriptable object implementation.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 10 years, 5 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
« no previous file with comments | « remoting/client/plugin/chromoting_scriptable_object.cc ('k') | remoting/client/x11_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/client/plugin/pepper_view.cc
===================================================================
--- remoting/client/plugin/pepper_view.cc (revision 53948)
+++ remoting/client/plugin/pepper_view.cc (working copy)
@@ -43,14 +43,12 @@
return;
}
- // TODO(ajwong): We're assuming the native format is BGRA_PREMUL below. This
- // is wrong.
- pp::ImageData image(pp::ImageData::GetNativeImageDataFormat(),
+ // TODO(ajwong): We shouldn't assume the image data format.
+ pp::ImageData image(PP_IMAGEDATAFORMAT_BGRA_PREMUL,
pp::Size(viewport_width_, viewport_height_),
false);
if (image.is_null()) {
- LOG(ERROR) << "Unable to allocate image of size: "
- << viewport_width_ << "x" << viewport_height_;
+ LOG(ERROR) << "Unable to allocate image.";
return;
}
« no previous file with comments | « remoting/client/plugin/chromoting_scriptable_object.cc ('k') | remoting/client/x11_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698