Frank Kobzik has posted comments on this change.
Change subject: frontend: Use console popup dialog in webadmin
......................................................................
Patch Set 1: (5 inline comments)
....................................................
File
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/presenter/popup/ConsolePopupPresenterWidget.java
Line 91: this.model = model;
Line 92: initView(model);
Line 93: initListeners(model);
Line 94:
Line 95: String vmName = model.getModel().getVM().getName();
Done
Line 96: getView().setVmName(vmName);
Line 97:
Line 98: super.init(model);
Line 99: }
....................................................
File
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/utils/ConsoleUtils.java
Line 19:
Line 20: private Boolean spiceAvailable;
Line 21: private Boolean rdpAvailable;
Line 22:
Line 23: private final String VNC_NOT_SUPPORTED_MESSAGE;
this will be merged in next patch of mine (it's on review now).
Line 24: private final String BROWSER_NOT_SUPPORTED_MESSAGE;
Line 25:
Line 26: private final ClientAgentType clientAgentType;
Line 27:
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/ConsoleAwareModel.java
Line 2:
Line 3: import org.ovirt.engine.core.common.businessentities.VM;
Line 4: import org.ovirt.engine.ui.uicommonweb.models.vms.ConsoleModel;
Line 5:
Line 6: public interface ConsoleAwareModel {
Done
Line 7:
Line 8: public boolean isPool();
Line 9:
Line 10: public ConsoleProtocol getSelectedProtocol();
....................................................
File
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabVirtualMachineView.java
Line 229: resources.consoleImage(),
resources.consoleDisabledImage()) {
Line 230: @Override
Line 231: protected UICommand resolveCommand() {
Line 232:
Line 233: UICommand command = new UICommand("ConsoleConnect",
new BaseCommandTarget() { // $NON-NLS-1$
Done
Line 234: @Override
Line 235: public void ExecuteCommand(UICommand uiCommand) {
Line 236: String errorMessage =
consoleManager.connectToConsole(getMainModel());
Line 237: if (errorMessage != null) {
Line 239: }
Line 240: }
Line 241: });
Line 242:
Line 243: command.setIsExecutionAllowed(
Done
Line 244: getMainModel().getSelectedItems() != null &&
getMainModel().getSelectedItems().size() == 1);
Line 245:
Line 246: return command;
Line 247: }
--
To view, visit http://gerrit.ovirt.org/12877
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I00158bbd8bd488dc527821b5f088fe598c9c2713
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Frank Kobzik <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Frank Kobzik <[email protected]>
Gerrit-Reviewer: Tomas Jelinek <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
