Skip to content

Commit

Permalink
correct javascript remote client ui executor to use new promise based…
Browse files Browse the repository at this point in the history
… execute introduced by 0fa587

Fixes SeleniumHQ#1521
  • Loading branch information
lukeis committed Jan 24, 2016
1 parent d1e9b27 commit cb6e9db
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions javascript/remote/ui/client.js
Original file line number Diff line number Diff line change
Expand Up @@ -172,8 +172,7 @@ remote.ui.Client.prototype.getSessionContainer = function() {
*/
remote.ui.Client.prototype.execute_ = function(command) {
this.banner_.setVisible(false);
var fn = goog.bind(this.executor_.execute, this.executor_, command);
return webdriver.promise.checkedNodeCall(fn).
return this.executor_.execute(command).
then(bot.response.checkResponse);
};

Expand Down Expand Up @@ -322,7 +321,7 @@ remote.ui.Client.prototype.onLoad_ = function(e) {
remote.ui.Client.prototype.onScreenshot_ = function() {
var session = this.sessionContainer_.getSelectedSession();
if (!session) {
goog.log.warning(this.log_,
goog.log.warning(this.log_,
'Cannot take screenshot; no session selected!');
return;
}
Expand Down

0 comments on commit cb6e9db

Please sign in to comment.