Index: ui/views/controls/native/native_view_host_wrapper.h |
diff --git a/ui/views/controls/native/native_view_host_wrapper.h b/ui/views/controls/native/native_view_host_wrapper.h |
index 3b3cb37fbdc0b5382b8819ca7223a0b4f88b28fd..5958757aba2d0ab7b73015589796a6d0a5319fe3 100644 |
--- a/ui/views/controls/native/native_view_host_wrapper.h |
+++ b/ui/views/controls/native/native_view_host_wrapper.h |
@@ -19,10 +19,10 @@ class VIEWS_EXPORT NativeViewHostWrapper { |
public: |
virtual ~NativeViewHostWrapper() {} |
- // Called right before a gfx::NativeView is attached to the associated |
- // NativeViewHost, allowing the wrapper to perform platform-specific |
- // initialization. |
- virtual void NativeViewWillAttach() = 0; |
+ // Called at the end of NativeViewHost::Attach, allowing the wrapper to |
+ // perform platform-specific operations that need to occur to complete |
+ // attaching the gfx::NativeView. |
+ virtual void AttachNativeView() = 0; |
// Called before the attached gfx::NativeView is detached from the |
// NativeViewHost, allowing the wrapper to perform platform-specific |
@@ -38,13 +38,18 @@ class VIEWS_EXPORT NativeViewHostWrapper { |
// rooted at a valid Widget. |
virtual void RemovedFromWidget() = 0; |
- // Installs a clip on the gfx::NativeView. |
+ // Installs a clip on the gfx::NativeView. These values are in the coordinate |
+ // space of the gfx::NativeView, so if this method is called from ShowWidget |
+ // then the values need to be translated. |
virtual void InstallClip(int x, int y, int w, int h) = 0; |
// Whether or not a clip has been installed on the wrapped gfx::NativeView. |
virtual bool HasInstalledClip() = 0; |
- // Removes the clip installed on the gfx::NativeView by way of InstallClip. |
+ // Removes the clip installed on the gfx::NativeView by way of InstallClip. A |
+ // following call to ShowWidget should occur after calling this method to |
+ // position the gfx::NativeView correctly, since the clipping process may have |
+ // adjusted its position. |
virtual void UninstallClip() = 0; |
// Shows the gfx::NativeView at the specified position (relative to the parent |