Decouple scheduling animation of webview plugins from layout.

third_party/WebKit/public/web/WebPluginContainer.h

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  * Copyright (C) 2014 Opera Software ASA. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
@@ skipping 47 matching lines @@
 
     // Returns the element containing this plugin.
     virtual WebElement element() = 0;
 
     virtual void dispatchProgressEvent(const WebString& type, bool lengthComputable, unsigned long long loaded, unsigned long long total, const WebString& url) = 0;
 
     virtual void invalidate() = 0;
     virtual void invalidateRect(const WebRect&) = 0;
     virtual void scrollRect(const WebRect&) = 0;
 
-    // Causes the container to be marked as needing layout, which in turn will cause
-    // layoutIfNeeded() to be called on any contained WebPlugin during the container's
-    // web view's lifecycle update, and in particular before calling paint() on the
-    // WebPlugin.
-    virtual void setNeedsLayout() = 0;
+    // Schedules an animation of the WebView that contains the plugin, as well as the plugin.
+    virtual void scheduleAnimation() = 0;

[wkorman, 2016/03/08 01:38:50]

If this is the way a new frame is requested, perhaps we should consider renaming
scheduleAnimation at some point, since as currently named I suspect most would
guess it is solely css animation related.

[chrishtr, 2016/03/10 19:06:14]

In some cases it isn't even running a frame, it's running async
events for script such as onresize. Still not exactly sure what changes
should be made to clean up frames vs events, I am intending to ask
around to the scheduling team.

 
     // Causes the container to report its current geometry via
     // WebPlugin::updateGeometry.
     virtual void reportGeometry() = 0;
 
     // Allow the plugin to pass script objects to the browser. The container
     // tracks ownership of script objects in order to allow browser references
     // to them to be dropped when clearScriptObjects is called.
     virtual void allowScriptObjects() = 0;
 
@@ skipping 61 matching lines @@
     // WebPluginContainer does *not* take ownership.
     virtual void setWebLayer(WebLayer*) = 0;
 
 protected:
     ~WebPluginContainer() { }
 };
 
 } // namespace blink
 
 #endif