Commit 6d4fa5f6 authored by sergeyu@chromium.org's avatar sergeyu@chromium.org

Fix click-to-play support in chromoting client

Click-to-play was broken when the plugin resizing code was
changed to update plugin style instead of width/height fields
directly. Problem is that 
remoting.ClientPlugin.showPluginForClickToPlay() was still using
trying to set width/height which are overridden by the CSS size.

NOTRY=true

Review URL: https://codereview.chromium.org/217353002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@260337 0039d316-1c4b-4281-b951-d872f2087c98
parent da67d5d7
......@@ -169,8 +169,8 @@ remoting.ClientPlugin.prototype.handleMessageMethod_ = function(message) {
if (message.method == 'hello') {
// Reset the size in case we had to enlarge it to support click-to-play.
this.plugin.width = 0;
this.plugin.height = 0;
this.plugin.style.width = '0px';
this.plugin.style.height = '0px';
this.pluginApiVersion_ = getNumberAttr(message.data, 'apiVersion');
this.pluginApiMinVersion_ = getNumberAttr(message.data, 'apiMinVersion');
......@@ -660,8 +660,8 @@ remoting.ClientPlugin.prototype.showPluginForClickToPlay_ = function() {
if (!this.helloReceived_) {
var width = 200;
var height = 200;
this.plugin.width = width;
this.plugin.height = height;
this.plugin.style.width = width + 'px';
this.plugin.style.height = height + 'px';
// Center the plugin just underneath the "Connnecting..." dialog.
var parentNode = this.plugin.parentNode;
var dialog = document.getElementById('client-dialog');
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment