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

Unified Diff: webkit/plugins/ppapi/ppapi_webplugin_impl.cc

Issue 7826017: Add PPB_Fullscreen;0.5. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 3 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: webkit/plugins/ppapi/ppapi_webplugin_impl.cc
===================================================================
--- webkit/plugins/ppapi/ppapi_webplugin_impl.cc (revision 99979)
+++ webkit/plugins/ppapi/ppapi_webplugin_impl.cc (working copy)
@@ -118,7 +118,8 @@
}
void WebPluginImpl::paint(WebCanvas* canvas, const WebRect& rect) {
- if (!instance_->IsFullscreenOrPending())
+ if (!instance_->IsFullscreenOrPending() &&
+ !instance_->IsFullscreenOrPending_Dev())
instance_->Paint(canvas, plugin_rect_, rect);
piman 2011/09/09 01:33:44 Maybe I'm confused about the implementation, but s
jeremya 2011/09/09 17:45:35 I think you're right -- in the old fullscreen mode
polina 2011/09/26 21:57:00 I did not have the full background on the differen
}
@@ -128,7 +129,8 @@
const WebVector<WebRect>& cut_outs_rects,
bool is_visible) {
plugin_rect_ = window_rect;
- if (!instance_->IsFullscreenOrPending())
+ if (!instance_->IsFullscreenOrPending() &&
+ !instance_->IsFullscreenOrPending_Dev())
instance_->ViewChanged(plugin_rect_, clip_rect);
}
@@ -145,7 +147,8 @@
bool WebPluginImpl::handleInputEvent(const WebKit::WebInputEvent& event,
WebKit::WebCursorInfo& cursor_info) {
- if (instance_->IsFullscreenOrPending())
+ if (instance_->IsFullscreenOrPending() ||
+ instance_->IsFullscreenOrPending_Dev())
return false;
return instance_->HandleInputEvent(event, &cursor_info);
}

Powered by Google App Engine
This is Rietveld 408576698