Frank Kobzik has uploaded a new change for review.

Change subject: frontend: Fixed default color depth for spice-activex
......................................................................

frontend: Fixed default color depth for spice-activex

This patch fixes setting default value of ColorDepth attribute of spice browser
plugin. For spice-xpi the value is "", whereas for spice-activex the value
is 0.

Change-Id: I58af05c0a2ff88dccb4634ec9a81dff84afe92a5
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=913183
Signed-off-by: Frantisek Kobzik <[email protected]>
---
M 
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/SpicePluginImpl.java
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/02/14502/1

diff --git 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/SpicePluginImpl.java
 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/SpicePluginImpl.java
index 6bb4c11..59154f0 100644
--- 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/SpicePluginImpl.java
+++ 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/SpicePluginImpl.java
@@ -167,7 +167,7 @@
                                                   client.ColorDepth = 
colorDepth;
                                                } else { //reset to defaults
                                                   client.DisableEffects = "";
-                                                  client.ColorDepth = "";
+                                                  client.ColorDepth = ""; // 
"" is default for spice-xpi
                                                }
 
                                                // only if the proxy is defined 
in VDC_OPTIONS
@@ -329,7 +329,7 @@
                                                        client.ColorDepth = 
colorDepth;
                                                    }else { //reset to defaults
                                                       client.DisableEffects = 
"";
-                                                      client.ColorDepth = "";
+                                                      client.ColorDepth = 0; 
// 0 is default for spice-activex
                                                    }
 
                                                    // only if the proxy is 
defined in VDC_OPTIONS


--
To view, visit http://gerrit.ovirt.org/14502
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I58af05c0a2ff88dccb4634ec9a81dff84afe92a5
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Frank Kobzik <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to