Skip to content

Commit 5820c57

Browse files
committed
Drivers don't return 'takesScreenshot' capability anymore, it's enabled by default.
1 parent 5457171 commit 5820c57

2 files changed

Lines changed: 1 addition & 24 deletions

File tree

javascript/remote/test/sessionview_test.html

Lines changed: 1 addition & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -96,27 +96,11 @@
9696
assertEquals(1, onDelete.getCallCount());
9797
}
9898

99-
function testCorrectlySetsScreenshotButtonStateBasedOnSessionSupport() {
100-
var session = new webdriver.Session('fluffy bunny', {});
101-
viewSession(session);
102-
assertFalse('Should be disabled', view.screenshotButton_.isEnabled());
103-
104-
session.getCapabilities().set('takesScreenshot', false);
105-
view.update(session);
106-
assertFalse('Should be disabled', view.screenshotButton_.isEnabled());
107-
108-
session.getCapabilities().set('takesScreenshot', true);
109-
view.update(session);
110-
assertTrue('Should be enabled', view.screenshotButton_.isEnabled());
111-
}
112-
11399
function testFiresScreenshotEventWhenButtonClicked() {
114100
var onScreenshot = goog.testing.recordFunction();
115101
goog.events.listen(view, remote.ui.Event.Type.SCREENSHOT, onScreenshot);
116102

117-
var session = new webdriver.Session('fluffy bunny', {
118-
'takesScreenshot': true
119-
});
103+
var session = new webdriver.Session('fluffy bunny', {});
120104
viewSession(session);
121105
bot.action.click(view.screenshotButton_.getElement());
122106
assertEquals(1, onScreenshot.getCallCount());

javascript/remote/ui/sessionview.js

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -212,13 +212,6 @@ remote.ui.SessionView.prototype.update = function(session) {
212212
if (session) {
213213
goog.dom.setTextContent(this.sessionIdSpan_, session.getId());
214214
this.capabilitiesTooltip_.update(session.getCapabilities());
215-
if (!session.getCapability('takesScreenshot')) {
216-
this.screenshotButton_.setEnabled(false);
217-
this.screenshotButton_.setTooltip('Screenshots not supported');
218-
} else {
219-
this.screenshotButton_.setEnabled(true);
220-
this.screenshotButton_.setTooltip('');
221-
}
222215
}
223216
};
224217

0 commit comments

Comments
 (0)