Skip to content

Commit 40377e4

Browse files
committed
java: Fixing sessionId format in marionette connector
1 parent bb13f6d commit 40377e4

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

java/client/src/org/openqa/selenium/firefox/internal/MarionetteConnection.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -192,9 +192,8 @@ public Response execute(Command command) throws IOException {
192192
Map<String, Object> map = new JsonToBeanConverter().convert(Map.class, rawResponse);
193193
Response response;
194194
if (DriverCommand.NEW_SESSION.equals(command.getName())) {
195-
// See https://bugzilla.mozilla.org/show_bug.cgi?id=1073732
196-
response = new Response(new SessionId(new BeanToJsonConverter().convert(map.get("value"))));
197-
response.setValue(Maps.newHashMap());
195+
response = new Response(new SessionId(map.get("sessionId").toString()));
196+
response.setValue(map.get("value"));
198197

199198
} else {
200199
if (map.containsKey("error")) {
@@ -307,7 +306,7 @@ private String serializeCommand(Command command) {
307306
map.put("name", commandName);
308307
if (command.getSessionId() != null) {
309308
// See https://bugzilla.mozilla.org/show_bug.cgi?id=1073732
310-
map.put("sessionId", new JsonToBeanConverter().convert(Map.class, command.getSessionId().toString()));
309+
map.put("sessionId", command.getSessionId().toString());
311310
}
312311
map.put("parameters", params);
313312

0 commit comments

Comments
 (0)