From be79f477e4d1e66e2d80e7bcd5f9689f3fbbcefc Mon Sep 17 00:00:00 2001 From: kelvinp Date: Wed, 8 Apr 2015 20:46:58 -0700 Subject: [Webapp Refactor] Clean up remoting.Clipboard. Currently, remoting.Clipboard is exposed as a global, such that it can hook events across different files, e.g. remoting.js, client_plugin_impl.js, application.js This CL moves clean up the file by: 1. Only create an instance of the clipboard when a session is connected. This removes the startSession() method on the clipboard. 2. Hook all events (copy/paste/focus changes) from within remoting.clipboard, which eliminates the need to have a global and effectively makes all its methods private. 3. Style cleanups, use single quotes for strings, make sure private variables are post-fixed with underscores. BUG=475317 Review URL: https://codereview.chromium.org/1078563002 Cr-Commit-Position: refs/heads/master@{#324354} --- remoting/webapp/crd/js/client_plugin.js | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) (limited to 'remoting/webapp/crd/js/client_plugin.js') diff --git a/remoting/webapp/crd/js/client_plugin.js b/remoting/webapp/crd/js/client_plugin.js index e8b3f8d..31bb60e 100644 --- a/remoting/webapp/crd/js/client_plugin.js +++ b/remoting/webapp/crd/js/client_plugin.js @@ -157,8 +157,14 @@ remoting.ClientPlugin.prototype.setConnectionEventHandler = * URL encoding the mouse cursor; the second and third parameters are * the cursor hotspot's x- and y-coordinates, respectively. */ -remoting.ClientPlugin.prototype.setMouseCursorHandler = - function(handler) {}; +remoting.ClientPlugin.prototype.setMouseCursorHandler = function(handler) {}; + +/** + * @param {function(string, string):void} handler Callback for processing + * clipboard data injected from the host. The first parameter is the mime + * type and the second parameter is the actual data. + */ +remoting.ClientPlugin.prototype.setClipboardHandler = function(handler) {}; /** * @param {function({rects:Array>}):void|null} handler Callback -- cgit v1.1