Chromium Code Reviews| Index: webkit/plugins/ppapi/ppapi_plugin_instance.h |
| diff --git a/webkit/plugins/ppapi/ppapi_plugin_instance.h b/webkit/plugins/ppapi/ppapi_plugin_instance.h |
| index 9949c174ae677761df29ade9115605cf182ec7fc..85c12b0f55fb37d96f4418436c19684ceecf427c 100644 |
| --- a/webkit/plugins/ppapi/ppapi_plugin_instance.h |
| +++ b/webkit/plugins/ppapi/ppapi_plugin_instance.h |
| @@ -63,7 +63,6 @@ struct PPP_Instance_Combined; |
| namespace webkit { |
| namespace ppapi { |
| -class FullscreenContainer; |
| class MessageChannel; |
| class ObjectVar; |
| class PluginDelegate; |
| @@ -242,10 +241,6 @@ class PluginInstance : public base::RefCounted<PluginInstance>, |
| // embedded in a page). |
| bool IsFullPagePlugin() const; |
| - FullscreenContainer* fullscreen_container() const { |
| - return fullscreen_container_; |
| - } |
| - |
| // FunctionGroupBase overrides. |
| virtual ::ppapi::thunk::PPB_Instance_FunctionAPI* AsPPB_Instance_FunctionAPI() |
| OVERRIDE; |
| @@ -467,10 +462,10 @@ class PluginInstance : public base::RefCounted<PluginInstance>, |
| // to use a more optimized painting path in some cases. |
| bool always_on_top_; |
| - // Plugin container for fullscreen mode. NULL if not in fullscreen mode. Note: |
| - // there is a transition state where fullscreen_container_ is non-NULL but |
| - // fullscreen_ is false (see above). |
| - FullscreenContainer* fullscreen_container_; |
| + // Since entering fullscreen mode is an asynchronous operation, we set this |
| + // variable to the desired state at the time we issue the fullscreen change |
| + // request. |
|
brettw
2011/08/24 23:14:04
It might be nice to mention here how you tell if t
jeremya
2011/08/24 23:40:45
Done.
|
| + bool desired_fullscreen_state_; |
| // True if we are in fullscreen mode. Note: it is false during the transition. |
| bool fullscreen_; |