Simplify computation of the invalidation rect for a frame.

third_party/WebKit/Source/web/WebPluginContainerImpl.h

Index: third_party/WebKit/Source/web/WebPluginContainerImpl.h
diff --git a/third_party/WebKit/Source/web/WebPluginContainerImpl.h b/third_party/WebKit/Source/web/WebPluginContainerImpl.h
index 131eb183a0bd4bc5ab57267e3e3b6dbdaed51d6c..e65e6913fefb41b41633619735f98d28232ed013 100644
--- a/third_party/WebKit/Source/web/WebPluginContainerImpl.h
+++ b/third_party/WebKit/Source/web/WebPluginContainerImpl.h
@@ -165,6 +165,10 @@ public:
 #endif
 
 private:
+    // Sets |clippedAbsoluteRect| to the absolute visible rect for the plugin, clipped to the visible screen of the root frame.

[chrishtr, 2015/11/03 19:22:55]

This is my understanding of what is intended after reading the old code. Is it
correct?

i.e. we want to compute the pixels in the root view of the document, bot clipped
to the visible viewport, and not clipped to it.

[tommycli, 2015/11/03 20:10:22]

Exactly. One is clipped to the visible viewport, the other pretends that the
user's monitor can show the whole webpage.

+    // If not null, sets |unclippedAbsoluteRect| tp the absolute visible rect for the plugin (but without also clipping to the screen).

[tommycli, 2015/11/03 20:10:22]

Tiny typo.

[chrishtr, 2015/11/03 21:24:56]

Fixed.

+    void computeClipRectsForPlugin(const HTMLFrameOwnerElement* pluginOwnerElement, IntRect& clippedAbsoluteRect, IntRect* unclippedAbsoluteRect) const;
+
     WebPluginContainerImpl(HTMLPlugInElement*, WebPlugin*);
     ~WebPluginContainerImpl() override;