Use LogLevelMapping to convert JSON wire protocol name to Level instance.

Without this, the java bindings fail to parse the "DEBUG" log level,
which is valid in JSON wire protocol but not a recognized Java log level
name.

Author: jleyba

java/client/src/org/openqa/selenium/remote/DesiredCapabilities.java

@@ -26,13 +26,13 @@
 
 import org.openqa.selenium.Capabilities;
 import org.openqa.selenium.Platform;
+import org.openqa.selenium.logging.LogLevelMapping;
 import org.openqa.selenium.logging.LoggingPreferences;
 
 import java.io.Serializable;
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.Map;
-import java.util.logging.Level;
 
 @SuppressWarnings("serial")
 public class DesiredCapabilities implements Serializable, Capabilities {
@@ -54,7 +54,7 @@ public DesiredCapabilities(Map<String, ?> rawMap) {
       Map<String, String> prefsMap = (Map<String, String>) rawMap.get(LOGGING_PREFS);
 
       for (String logType : prefsMap.keySet()) {
-        prefs.enable(logType, Level.parse(prefsMap.get(logType)));
+        prefs.enable(logType, LogLevelMapping.toLevel(prefsMap.get(logType)));
       }
       capabilities.put(LOGGING_PREFS, prefs);
       // So it does not get added twice

java/client/test/org/openqa/selenium/remote/DesiredCapabilitiesTest.java

@@ -24,11 +24,15 @@
 import org.openqa.selenium.Capabilities;
 import org.openqa.selenium.StubDriver;
 import org.openqa.selenium.WebDriver;
+import org.openqa.selenium.logging.LoggingPreferences;
 
+import java.util.HashMap;
 import java.util.Map;
 import java.util.concurrent.ConcurrentHashMap;
+import java.util.logging.Level;
 
 import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertSame;
 
 @RunWith(JUnit4.class)
 public class DesiredCapabilitiesTest {
@@ -75,4 +79,18 @@ public void testCopyConstructorDoesNotAliasToArgument() {
     assertEquals("ie", origCapabilities.getCapability("Browser"));
     assertEquals("firefox", newCapabilities.getCapability("Browser"));
   }
+
+  @Test
+  public void testExtractDebugLogLevelFromCapabilityMap() {
+    Map<String, Object> capabilitiesMap = new HashMap<String, Object>() {{
+      put(CapabilityType.LOGGING_PREFS, new HashMap<String, String>() {{
+        put("browser", "DEBUG");
+      }});
+    }};
+
+    DesiredCapabilities caps = new DesiredCapabilities(capabilitiesMap);
+    LoggingPreferences prefs =
+        (LoggingPreferences) caps.getCapability(CapabilityType.LOGGING_PREFS);
+    assertSame(Level.FINE, prefs.getLevel("browser"));
+  }
 }