Minor java 8ism: optional.ifPresent, not an if statement

shs96c · shs96c · commit 0020080868db · 2018-04-12T18:48:30.000-07:00
diff --git a/java/server/src/org/openqa/selenium/remote/server/rest/Responses.java b/java/server/src/org/openqa/selenium/remote/server/rest/Responses.java
@@ -84,7 +84,7 @@ public static Response failure(
 
     if (reason != null) {
       JsonObject json = new Json().toJsonElement(reason).getAsJsonObject();
-      json.addProperty("screen", screenshot.orElse(null));
+      screenshot.ifPresent(screen -> json.addProperty("screen", screen));
       response.setValue(json);
     }
     return response;

Original file line number	Diff line number	Diff line change
`@@ -84,7 +84,7 @@ public static Response failure(`
`84`	`84`
`85`	`85`	`if (reason != null) {`
`86`	`86`	`JsonObject json = new Json().toJsonElement(reason).getAsJsonObject();`
`87`		`- json.addProperty("screen", screenshot.orElse(null));`
	`87`	`+ screenshot.ifPresent(screen -> json.addProperty("screen", screen));`
`88`	`88`	`response.setValue(json);`
`89`	`89`	`}`
`90`	`90`	`return response;`