No need to create another in-memory output stream, CommandLine already has one.

Author: barancev

java/client/src/org/openqa/selenium/firefox/FirefoxBinary.java

@@ -326,11 +326,10 @@ public void waitFor(long timeout) throws InterruptedException, IOException {
    * @throws IOException IO exception reading from the output stream of the firefox process
    */
   public String getConsoleOutput() throws IOException {
-    if (process == null || stream == null) {
+    if (process == null) {
       return null;
     }
-
-    return stream instanceof CircularOutputStream ? stream.toString() : null;
+    return process.getStdOut();
   }
 
   public long getTimeout() {
@@ -358,13 +357,13 @@ public void quit() {
 
   private OutputStream getDefaultOutputStream() throws FileNotFoundException {
     String firefoxLogFile = System.getProperty(FirefoxDriver.SystemProperty.BROWSER_LOGFILE);
-    if (firefoxLogFile == null) {
-      return new CircularOutputStream();
-    }
-    if ("/dev/stdout".equals(firefoxLogFile)) {
-      return System.out;
+    if (firefoxLogFile != null) {
+      if ("/dev/stdout".equals(firefoxLogFile)) {
+        return System.out;
+      }
+      return new FileOutputStream(firefoxLogFile);
     }
-    return new MultiOutputStream(new CircularOutputStream(), new FileOutputStream(firefoxLogFile));
+    return null;
   }
 
   /**