returned basic ability of recording screen video

Refactored QuickTime implementation hardly to be more maintainable
Added UI button for staring/stopping recording.

Author: xSAVIKx

README.md

@@ -27,6 +27,7 @@ This project gives the opportunity to use your phone even with a broken screen.
 - Write messages using PC keyboard
 - Support for landscape mode
 - Browse your phone files on PC
+- Record video of your phone screen while browsing
 
 [Small wiki of project][wiki]
 

docs/index.md

@@ -17,6 +17,7 @@ This project gives the opportunity to use your phone even with a broken screen.
 - Write messages using PC keyboard
 - Support for landscape mode
 - Browse your phone files on PC
+- Record video of your phone screen while browsing
 
 [Small wiki of project][wiki]
 

pom.xml

@@ -9,6 +9,7 @@
         <spring.version>4.3.5.RELEASE</spring.version>
         <ddmlib.version>25.2.0</ddmlib.version>
         <log4j.version>1.2.17</log4j.version>
+        <guava.version>21.0</guava.version>
         <main.class>com.github.xsavikx.androidscreencast.Main</main.class>
         <jdk.version>1.8</jdk.version>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
@@ -43,6 +44,11 @@
             <artifactId>spring-context</artifactId>
             <version>${spring.version}</version>
         </dependency>
+        <dependency>
+            <groupId>com.google.guava</groupId>
+            <artifactId>guava</artifactId>
+            <version>${guava.version}</version>
+        </dependency>
     </dependencies>
     <build>
         <plugins>
@@ -167,5 +173,6 @@
         Write messages using PC keyboard
         Support for landscape mode
         Browse your phone files on PC
+        Record video of your phone screen while browsing
     </description>
 </project>

src/main/java/com/github/xsavikx/androidscreencast/api/injector/ScreenCaptureRunnable.java

@@ -74,8 +74,8 @@ private RawImage getScreenshot() throws InterruptedException, ClosedByInterruptE
             }
             currentAdbCommandTimeout = defaultAdbCommandTimeout;
         } catch (TimeoutException e) {
-            LOGGER.warn("Adb command timeout happened. Timeout would be increased by 1 second for the next try.", e);
             currentAdbCommandTimeout++;
+            LOGGER.warn(String.format("Adb command timeout happened. Timeout would be set to %d for the next try.", currentAdbCommandTimeout), e);
         } catch (AdbCommandRejectedException e) {
             LOGGER.warn("ADB Command was rejected. Will try again in 100 ms.");
             Thread.sleep(100);
@@ -99,7 +99,7 @@ private void display(RawImage rawImage) {
             SwingUtilities.invokeLater(() -> {
                 try {
                     qos.writeFrame(image, FRAME_DURATION);
-                } catch (IOException e) {
+                } catch (IORuntimeException e) {
                     LOGGER.error(e);
                 }
             });
@@ -125,13 +125,9 @@ public void startRecording(File file) {
     }
 
     public void stopRecording() {
-        try {
-            QuickTimeOutputStream o = qos;
-            qos = null;
-            o.close();
-        } catch (IOException e) {
-            throw new IORuntimeException(e);
-        }
+        QuickTimeOutputStream o = qos;
+        qos = null;
+        o.close();
     }
 
     public void toggleOrientation() {

src/main/java/com/github/xsavikx/androidscreencast/api/recording/DataAtomOutputStream.java

@@ -1,5 +1,8 @@
 package com.github.xsavikx.androidscreencast.api.recording;
 
+import com.github.xsavikx.androidscreencast.api.recording.atom.AtomType;
+import com.github.xsavikx.androidscreencast.exception.IORuntimeException;
+
 import java.io.FilterOutputStream;
 import java.io.IOException;
 import java.io.OutputStream;
@@ -8,6 +11,9 @@
 import java.util.Date;
 import java.util.GregorianCalendar;
 
+import static com.google.common.base.Preconditions.checkArgument;
+import static com.google.common.base.Preconditions.checkNotNull;
+
 public class DataAtomOutputStream extends FilterOutputStream {
 
     protected static final long MAC_TIMESTAMP_EPOCH = new GregorianCalendar(1904, Calendar.JANUARY, 1).getTimeInMillis();
@@ -279,21 +285,28 @@ public void writeShort(int v) throws IOException {
     /**
      * Writes an Atom Type identifier (4 bytes).
      *
-     * @param s A string with a length of 4 characters.
+     * @param type A string with a length of 4 characters.
      */
-    public void writeType(String s) throws IOException {
-        if (s.length() != 4) {
-            throw new IllegalArgumentException("type string must have 4 characters");
-        }
-
+    private void writeType(String type) throws IOException {
+        checkNotNull(type, "Type string should not be null");
+        checkArgument(type.length() == 4, "Type string must have exactly 4 characters. type=%s", type);
         try {
-            out.write(s.getBytes("ASCII"), 0, 4);
+            out.write(type.getBytes("ASCII"), 0, 4);
             incCount(4);
         } catch (UnsupportedEncodingException e) {
             throw new InternalError(e.toString());
         }
     }
 
+    /**
+     * Writes an Atom Type identifier (4 bytes).
+     *
+     * @param atomType AtomType to be written
+     */
+    public void writeType(AtomType atomType) throws IOException {
+        writeType(atomType.toStringRepresentation());
+    }
+
     /**
      * Writes an unsigned 32 bit integer value.
      *
@@ -314,4 +327,13 @@ public void writeUShort(int v) throws IOException {
         incCount(2);
     }
 
+    @Override
+    public void close() {
+        try {
+            flush();
+        } catch (IOException e) {
+            throw new IORuntimeException(e);
+        }
+    }
+
 }