Improve importing progress reporting
authorNiki Roo <niki@nikiroo.be>
Sun, 5 Mar 2017 22:18:51 +0000 (23:18 +0100)
committerNiki Roo <niki@nikiroo.be>
Sun, 5 Mar 2017 22:18:51 +0000 (23:18 +0100)
- We know report more information, and more often

src/be/nikiroo/fanfix/supported/BasicSupport.java
src/be/nikiroo/fanfix/supported/Cbz.java
src/be/nikiroo/fanfix/supported/E621.java
src/be/nikiroo/fanfix/supported/Epub.java
src/be/nikiroo/fanfix/supported/Fanfiction.java
src/be/nikiroo/fanfix/supported/Fimfiction.java
src/be/nikiroo/fanfix/supported/MangaFox.java
src/be/nikiroo/fanfix/supported/Text.java
src/be/nikiroo/fanfix/supported/YiffStar.java
src/be/nikiroo/fanfix/test/BasicSupportTest.java

index 18113188b8ac358b19af20ff62a845a91d7be9b1..97338ba4b1d8fca449c16b5799e5d4d8a23b13ed 100644 (file)
@@ -216,6 +216,8 @@ public abstract class BasicSupport {
         *            the source of the story
         * @param in
         *            the input (the main resource)
+        * @param pg
+        *            the optional progress reporter
         * 
         * @return the chapters
         * 
@@ -223,7 +225,7 @@ public abstract class BasicSupport {
         *             in case of I/O error
         */
        protected abstract List<Entry<String, URL>> getChapters(URL source,
-                       InputStream in) throws IOException;
+                       InputStream in, Progress pg) throws IOException;
 
        /**
         * Return the content of the chapter (possibly HTML encoded, if
@@ -235,6 +237,8 @@ public abstract class BasicSupport {
         *            the input (the main resource)
         * @param number
         *            the chapter number
+        * @param pg
+        *            the optional progress reporter
         * 
         * @return the content
         * 
@@ -242,7 +246,7 @@ public abstract class BasicSupport {
         *             in case of I/O error
         */
        protected abstract String getChapterContent(URL source, InputStream in,
-                       int number) throws IOException;
+                       int number, Progress pg) throws IOException;
 
        /**
         * Log into the support (can be a no-op depending upon the support).
@@ -298,7 +302,7 @@ public abstract class BasicSupport {
         *             in case of I/O error
         */
        public Story processMeta(URL url) throws IOException {
-               return processMeta(url, true, false);
+               return processMeta(url, true, false, null);
        }
 
        /**
@@ -310,15 +314,24 @@ public abstract class BasicSupport {
         * 
         * @param close
         *            close "this" and "in" when done
+        * @param pg
+        *            the optional progress reporter
         * 
         * @return the {@link Story}
         * 
         * @throws IOException
         *             in case of I/O error
         */
-       protected Story processMeta(URL url, boolean close, boolean getDesc)
-                       throws IOException {
+       protected Story processMeta(URL url, boolean close, boolean getDesc,
+                       Progress pg) throws IOException {
+               if (pg == null) {
+                       pg = new Progress();
+               } else {
+                       pg.setMinMax(0, 100);
+               }
+
                login();
+               pg.setProgress(10);
 
                url = getCanonicalUrl(url);
 
@@ -331,6 +344,7 @@ public abstract class BasicSupport {
 
                try {
                        preprocess(url, getInput());
+                       pg.setProgress(30);
 
                        Story story = new Story();
                        MetaData meta = getMeta(url, getInput());
@@ -340,18 +354,23 @@ public abstract class BasicSupport {
                        }
                        story.setMeta(meta);
 
+                       pg.setProgress(50);
+
                        if (meta != null && meta.getCover() == null) {
                                meta.setCover(getDefaultCover(meta.getSubject()));
                        }
 
+                       pg.setProgress(60);
+
                        if (getDesc) {
                                String descChapterName = Instance.getTrans().getString(
                                                StringId.DESCRIPTION);
                                story.getMeta().setResume(
                                                makeChapter(url, 0, descChapterName,
-                                                               getDesc(url, getInput())));
+                                                               getDesc(url, getInput()), null));
                        }
 
+                       pg.setProgress(100);
                        return story;
                } finally {
                        if (close) {
@@ -394,10 +413,15 @@ public abstract class BasicSupport {
                url = getCanonicalUrl(url);
                pg.setProgress(1);
                try {
-                       Story story = processMeta(url, false, true);
-                       pg.setProgress(10);
+                       Progress pgMeta = new Progress();
+                       pg.addProgress(pgMeta, 10);
+                       Story story = processMeta(url, false, true, pgMeta);
+                       if (!pgMeta.isDone()) {
+                               pgMeta.setProgress(pgMeta.getMax()); // 10%
+                       }
+
                        if (story == null) {
-                               pg.setProgress(100);
+                               pg.setProgress(90);
                                return null;
                        }
 
@@ -405,24 +429,47 @@ public abstract class BasicSupport {
 
                        setCurrentReferer(url);
 
+                       Progress pgGetChapters = new Progress();
+                       pg.addProgress(pgGetChapters, 10);
                        story.setChapters(new ArrayList<Chapter>());
+                       List<Entry<String, URL>> chapters = getChapters(url, getInput(),
+                                       pgGetChapters);
+                       if (!pgGetChapters.isDone()) {
+                               pgGetChapters.setProgress(pgGetChapters.getMax()); // 20%
+                       }
 
-                       List<Entry<String, URL>> chapters = getChapters(url, getInput());
-                       pg.setProgress(20);
-
-                       int i = 1;
                        if (chapters != null) {
-                               Progress pgChaps = new Progress(0, chapters.size());
+                               Progress pgChaps = new Progress("Extracting chapters", 0,
+                                               chapters.size() * 300);
                                pg.addProgress(pgChaps, 80);
 
                                long words = 0;
+                               int i = 1;
                                for (Entry<String, URL> chap : chapters) {
+                                       pgChaps.setName("Extracting chapter " + i);
                                        setCurrentReferer(chap.getValue());
                                        InputStream chapIn = Instance.getCache().open(
                                                        chap.getValue(), this, true);
+                                       pgChaps.setProgress(i * 100);
                                        try {
+                                               Progress pgGetChapterContent = new Progress();
+                                               Progress pgMakeChapter = new Progress();
+                                               pgChaps.addProgress(pgGetChapterContent, 100);
+                                               pgChaps.addProgress(pgMakeChapter, 100);
+
+                                               String content = getChapterContent(url, chapIn, i,
+                                                               pgGetChapterContent);
+                                               if (!pgGetChapterContent.isDone()) {
+                                                       pgGetChapterContent.setProgress(pgGetChapterContent
+                                                                       .getMax());
+                                               }
+
                                                Chapter cc = makeChapter(url, i, chap.getKey(),
-                                                               getChapterContent(url, chapIn, i));
+                                                               content, pgMakeChapter);
+                                               if (!pgMakeChapter.isDone()) {
+                                                       pgMakeChapter.setProgress(pgMakeChapter.getMax());
+                                               }
+
                                                words += cc.getWords();
                                                story.getChapters().add(cc);
                                                if (story.getMeta() != null) {
@@ -432,10 +479,12 @@ public abstract class BasicSupport {
                                                chapIn.close();
                                        }
 
-                                       pgChaps.setProgress(i++);
+                                       i++;
                                }
+
+                               pgChaps.setName("Extracting chapters");
                        } else {
-                               pg.setProgress(100);
+                               pg.setProgress(80);
                        }
 
                        return story;
@@ -531,6 +580,8 @@ public abstract class BasicSupport {
         *            the chapter name
         * @param content
         *            the chapter content
+        * @param pg
+        *            the optional progress reporter
         * 
         * @return the {@link Chapter}
         * 
@@ -538,7 +589,7 @@ public abstract class BasicSupport {
         *             in case of I/O error
         */
        protected Chapter makeChapter(URL source, int number, String name,
-                       String content) throws IOException {
+                       String content, Progress pg) throws IOException {
                // Chapter name: process it correctly, then remove the possible
                // redundant "Chapter x: " in front of it
                String chapterName = processPara(name).getContent().trim();
@@ -566,7 +617,7 @@ public abstract class BasicSupport {
                Chapter chap = new Chapter(number, chapterName);
 
                if (content != null) {
-                       List<Paragraph> paras = makeParagraphs(source, content);
+                       List<Paragraph> paras = makeParagraphs(source, content, pg);
                        long words = 0;
                        for (Paragraph para : paras) {
                                words += para.getWords();
@@ -586,14 +637,20 @@ public abstract class BasicSupport {
         *            the source URL of the story
         * @param content
         *            the textual content
+        * @param pg
+        *            the optional progress reporter
         * 
         * @return the {@link Paragraph}s
         * 
         * @throws IOException
         *             in case of I/O error
         */
-       protected List<Paragraph> makeParagraphs(URL source, String content)
-                       throws IOException {
+       protected List<Paragraph> makeParagraphs(URL source, String content,
+                       Progress pg) throws IOException {
+               if (pg == null) {
+                       pg = new Progress();
+               }
+
                if (isHtml()) {
                        // Special <HR> processing:
                        content = content.replaceAll("(<hr [^>]*>)|(<hr/>)|(<hr>)",
@@ -604,10 +661,19 @@ public abstract class BasicSupport {
 
                if (content != null && !content.trim().isEmpty()) {
                        if (isHtml()) {
-                               for (String line : content.split("(<p>|</p>|<br>|<br/>)")) {
+                               String[] tab = content.split("(<p>|</p>|<br>|<br/>)");
+                               pg.setMinMax(0, tab.length);
+                               int i = 1;
+                               for (String line : tab) {
+                                       if (line.startsWith("[") && line.endsWith("]")) {
+                                               pg.setName("Extracting image " + i);
+                                       }
                                        paras.add(makeParagraph(source, line.trim()));
+                                       pg.setProgress(i++);
                                }
+                               pg.setName(null);
                        } else {
+                               List<String> lines = new ArrayList<String>();
                                BufferedReader buff = null;
                                try {
                                        buff = new BufferedReader(
@@ -615,13 +681,24 @@ public abstract class BasicSupport {
                                                                        content.getBytes("UTF-8")), "UTF-8"));
                                        for (String line = buff.readLine(); line != null; line = buff
                                                        .readLine()) {
-                                               paras.add(makeParagraph(source, line.trim()));
+                                               lines.add(line.trim());
                                        }
                                } finally {
                                        if (buff != null) {
                                                buff.close();
                                        }
                                }
+
+                               pg.setMinMax(0, lines.size());
+                               int i = 0;
+                               for (String line : lines) {
+                                       if (line.startsWith("[") && line.endsWith("]")) {
+                                               pg.setName("Extracting image " + i);
+                                       }
+                                       paras.add(makeParagraph(source, line));
+                                       pg.setProgress(i++);
+                               }
+                               pg.setName(null);
                        }
 
                        // Check quotes for "bad" format
index 7113248ba485fb4b91ad06f5b614cd49f1b7a3c7..b041b6d468a30a81683465bef8968dd35c9c56e0 100644 (file)
@@ -65,14 +65,19 @@ class Cbz extends Epub {
                        pg.setMinMax(0, 100);
                }
 
-               Story story = processMeta(url, false, true);
+               Progress pgMeta = new Progress();
+               pg.addProgress(pgMeta, 10);
+               Story story = processMeta(url, false, true, pgMeta);
+               if (!pgMeta.isDone()) {
+                       pgMeta.setProgress(pgMeta.getMax()); // 10%
+               }
+
                story.setChapters(new ArrayList<Chapter>());
                Chapter chap = new Chapter(1, null);
                story.getChapters().add(chap);
 
                ZipInputStream zipIn = new ZipInputStream(getInput());
 
-               pg.setProgress(10);
                List<String> images = new ArrayList<String>();
                for (ZipEntry entry = zipIn.getNextEntry(); entry != null; entry = zipIn
                                .getNextEntry()) {
index 05d122e68b89efc71587fda628f9b17e720736d7..527e0928cc33cb51215d30ca505dcd6dd4174e8d 100644 (file)
@@ -90,7 +90,7 @@ class E621 extends BasicSupport {
        private BufferedImage getCover(URL source) throws IOException {
                InputStream in = Instance.getCache().open(source, this, true);
                String images = getChapterContent(new URL(source.toString() + "?page="
-                               + 1), in, 1);
+                               + 1), in, 1, null);
                if (!images.isEmpty()) {
                        int pos = images.indexOf("<br/>");
                        if (pos >= 0) {
@@ -191,8 +191,8 @@ class E621 extends BasicSupport {
        }
 
        @Override
-       protected List<Entry<String, URL>> getChapters(URL source, InputStream in)
-                       throws IOException {
+       protected List<Entry<String, URL>> getChapters(URL source, InputStream in,
+                       Progress pg) throws IOException {
                List<Entry<String, URL>> urls = new ArrayList<Entry<String, URL>>();
                int last = 1; // no pool/show when only one page
 
@@ -240,8 +240,8 @@ class E621 extends BasicSupport {
        }
 
        @Override
-       protected String getChapterContent(URL source, InputStream in, int number)
-                       throws IOException {
+       protected String getChapterContent(URL source, InputStream in, int number,
+                       Progress pg) throws IOException {
                StringBuilder builder = new StringBuilder();
                String staticSite = "https://static1.e621.net";
                if (source.getHost().contains("e926")) {
index 437197a5a66cb12cee19a05ef192e9d35622494d..47da1ac2259abbb313788583ba4420ba1dae8134 100644 (file)
@@ -16,6 +16,7 @@ import be.nikiroo.fanfix.Instance;
 import be.nikiroo.fanfix.data.MetaData;
 import be.nikiroo.utils.IOUtils;
 import be.nikiroo.utils.MarkableFileInputStream;
+import be.nikiroo.utils.Progress;
 
 /**
  * Support class for EPUB files created with this program (as we need some
@@ -60,22 +61,22 @@ class Epub extends InfoText {
        }
 
        @Override
-       protected List<Entry<String, URL>> getChapters(URL source, InputStream in)
-                       throws IOException {
+       protected List<Entry<String, URL>> getChapters(URL source, InputStream in,
+                       Progress pg) throws IOException {
                if (fakeIn != null) {
                        fakeIn.reset();
-                       return super.getChapters(fakeSource, fakeIn);
+                       return super.getChapters(fakeSource, fakeIn, pg);
                }
 
                return null;
        }
 
        @Override
-       protected String getChapterContent(URL source, InputStream in, int number)
-                       throws IOException {
+       protected String getChapterContent(URL source, InputStream in, int number,
+                       Progress pg) throws IOException {
                if (fakeIn != null) {
                        fakeIn.reset();
-                       return super.getChapterContent(fakeSource, fakeIn, number);
+                       return super.getChapterContent(fakeSource, fakeIn, number, pg);
                }
 
                return null;
index 90736567adbd70a43ae2afc16ca42671424be04e..1d1f3f4a09cbbd88939c762434a9d57a0cfd342c 100644 (file)
@@ -15,6 +15,7 @@ import java.util.Scanner;
 import be.nikiroo.fanfix.Instance;
 import be.nikiroo.fanfix.bundles.Config;
 import be.nikiroo.fanfix.data.MetaData;
+import be.nikiroo.utils.Progress;
 import be.nikiroo.utils.StringUtils;
 
 /**
@@ -210,7 +211,8 @@ class Fanfiction extends BasicSupport {
        }
 
        @Override
-       protected List<Entry<String, URL>> getChapters(URL source, InputStream in) {
+       protected List<Entry<String, URL>> getChapters(URL source, InputStream in,
+                       Progress pg) {
                List<Entry<String, URL>> urls = new ArrayList<Entry<String, URL>>();
 
                String base = source.toString();
@@ -286,7 +288,8 @@ class Fanfiction extends BasicSupport {
        }
 
        @Override
-       protected String getChapterContent(URL source, InputStream in, int number) {
+       protected String getChapterContent(URL source, InputStream in, int number,
+                       Progress pg) {
                StringBuilder builder = new StringBuilder();
                String startAt = "class='storytext ";
                String endAt1 = "function review_init";
index 377e369e1a506cf4eb928dfc793772ab8a46292e..6293c1147f5530a058d59381bf9fb153aebf660f 100644 (file)
@@ -14,6 +14,7 @@ import java.util.Scanner;
 
 import be.nikiroo.fanfix.Instance;
 import be.nikiroo.fanfix.data.MetaData;
+import be.nikiroo.utils.Progress;
 import be.nikiroo.utils.StringUtils;
 
 /**
@@ -176,7 +177,8 @@ class Fimfiction extends BasicSupport {
        }
 
        @Override
-       protected List<Entry<String, URL>> getChapters(URL source, InputStream in) {
+       protected List<Entry<String, URL>> getChapters(URL source, InputStream in,
+                       Progress pg) {
                List<Entry<String, URL>> urls = new ArrayList<Entry<String, URL>>();
                @SuppressWarnings("resource")
                Scanner scan = new Scanner(in, "UTF-8");
@@ -232,7 +234,8 @@ class Fimfiction extends BasicSupport {
        }
 
        @Override
-       protected String getChapterContent(URL source, InputStream in, int number) {
+       protected String getChapterContent(URL source, InputStream in, int number,
+                       Progress pg) {
                return getLine(in, "<div id=\"chapter_container\">", 1);
        }
 
index d86cc738c2f731707e98bb328e93f4f2005a134e..a379e877ecb5c5d296b85708b0b48ac89e856716 100644 (file)
@@ -14,6 +14,7 @@ import java.util.Scanner;
 import be.nikiroo.fanfix.Instance;
 import be.nikiroo.fanfix.data.MetaData;
 import be.nikiroo.utils.IOUtils;
+import be.nikiroo.utils.Progress;
 import be.nikiroo.utils.StringUtils;
 
 class MangaFox extends BasicSupport {
@@ -197,7 +198,8 @@ class MangaFox extends BasicSupport {
        }
 
        @Override
-       protected List<Entry<String, URL>> getChapters(URL source, InputStream in) {
+       protected List<Entry<String, URL>> getChapters(URL source, InputStream in,
+                       Progress pg) {
                List<Entry<String, URL>> urls = new ArrayList<Entry<String, URL>>();
 
                String volumeAt = "<h3 class=\"volume\">";
@@ -240,9 +242,6 @@ class MangaFox extends BasicSupport {
                                        }
                                }
 
-                               // to help with the retry and the originalUrl
-                               refresh(url);
-
                                try {
                                        final String key = name;
                                        final URL value = new URL(url);
@@ -265,6 +264,19 @@ class MangaFox extends BasicSupport {
                        }
                }
 
+               if (pg == null) {
+                       pg = new Progress(0, urls.size());
+               } else {
+                       pg.setMinMax(0, urls.size());
+               }
+
+               int i = 1;
+               for (Entry<String, URL> entry : urls) {
+                       // to help with the retry and the originalUrl
+                       refresh(entry.getValue().toString());
+                       pg.setProgress(i++);
+               }
+
                // the chapters are in reversed order
                Collections.reverse(urls);
 
@@ -272,12 +284,22 @@ class MangaFox extends BasicSupport {
        }
 
        @Override
-       protected String getChapterContent(URL source, InputStream in, int number) {
+       protected String getChapterContent(URL source, InputStream in, int number,
+                       Progress pg) {
+               if (pg == null) {
+                       pg = new Progress();
+               } else {
+                       // Since we have no idea how many images we have, we cycle from 0
+                       // to max, then again, then again...
+                       pg.setMinMax(0, 20);
+               }
+
                StringBuilder builder = new StringBuilder();
                String base = getCurrentReferer().toString();
                int pos = base.lastIndexOf('/');
                base = base.substring(0, pos + 1); // including the '/' at the end
 
+               int i = 1;
                boolean close = false;
                while (in != null) {
                        String linkNextLine = getLine(in, "return enlarge()", 0);
@@ -317,6 +339,7 @@ class MangaFox extends BasicSupport {
 
                        // to help with the retry and the originalUrl, part 2
                        refresh(linkImage);
+                       pg.setProgress((i++) % pg.getMax());
 
                        if (close) {
                                try {
@@ -333,6 +356,7 @@ class MangaFox extends BasicSupport {
                                        url = new URL(base + linkNext);
                                        in = openEx(base + linkNext);
                                        setCurrentReferer(url);
+                                       pg.setProgress((i++) % pg.getMax());
                                } catch (IOException e) {
                                        Instance.syserr(new IOException(
                                                        "Cannot get the next manga page which is: "
index 3b486ce8d0c4a36b8610a3bd10da9943e3dbfb6c..67156956ee14d1389197d02b888b02b1f4a34336 100644 (file)
@@ -14,6 +14,7 @@ import java.util.Scanner;
 import be.nikiroo.fanfix.Instance;
 import be.nikiroo.fanfix.bundles.Config;
 import be.nikiroo.fanfix.data.MetaData;
+import be.nikiroo.utils.Progress;
 
 /**
  * Support class for local stories encoded in textual format, with a few rules:
@@ -147,7 +148,7 @@ class Text extends BasicSupport {
 
        @Override
        protected String getDesc(URL source, InputStream in) throws IOException {
-               return getChapterContent(source, in, 0);
+               return getChapterContent(source, in, 0, null);
        }
 
        private BufferedImage getCover(URL source) throws IOException {
@@ -169,8 +170,8 @@ class Text extends BasicSupport {
        }
 
        @Override
-       protected List<Entry<String, URL>> getChapters(URL source, InputStream in)
-                       throws IOException {
+       protected List<Entry<String, URL>> getChapters(URL source, InputStream in,
+                       Progress pg) throws IOException {
                List<Entry<String, URL>> chaps = new ArrayList<Entry<String, URL>>();
                @SuppressWarnings("resource")
                Scanner scan = new Scanner(in, "UTF-8");
@@ -208,8 +209,8 @@ class Text extends BasicSupport {
        }
 
        @Override
-       protected String getChapterContent(URL source, InputStream in, int number)
-                       throws IOException {
+       protected String getChapterContent(URL source, InputStream in, int number,
+                       Progress pg) throws IOException {
                StringBuilder builder = new StringBuilder();
                @SuppressWarnings("resource")
                Scanner scan = new Scanner(in, "UTF-8");
index ed75b10ef4e7e290276d599f6c591f4fc78ef6e6..a2126db5fcfc55d991a336f9111b2430f9427e32 100644 (file)
@@ -15,6 +15,7 @@ import java.util.Scanner;
 import be.nikiroo.fanfix.Instance;
 import be.nikiroo.fanfix.bundles.Config;
 import be.nikiroo.fanfix.data.MetaData;
+import be.nikiroo.utils.Progress;
 import be.nikiroo.utils.StringUtils;
 
 /**
@@ -130,7 +131,7 @@ class YiffStar extends BasicSupport {
        private BufferedImage getCover(URL source, InputStream in)
                        throws IOException {
 
-               List<Entry<String, URL>> chaps = getChapters(source, in);
+               List<Entry<String, URL>> chaps = getChapters(source, in, null);
                if (!chaps.isEmpty()) {
                        in = Instance.getCache().open(chaps.get(0).getValue(), this, true);
                        String line = getLine(in, " name=\"og:image\"", 0);
@@ -185,8 +186,8 @@ class YiffStar extends BasicSupport {
        }
 
        @Override
-       protected List<Entry<String, URL>> getChapters(URL source, InputStream in)
-                       throws IOException {
+       protected List<Entry<String, URL>> getChapters(URL source, InputStream in,
+                       Progress pg) throws IOException {
                List<Entry<String, URL>> urls = new ArrayList<Entry<String, URL>>();
 
                @SuppressWarnings("resource")
@@ -225,8 +226,8 @@ class YiffStar extends BasicSupport {
        }
 
        @Override
-       protected String getChapterContent(URL source, InputStream in, int number)
-                       throws IOException {
+       protected String getChapterContent(URL source, InputStream in, int number,
+                       Progress pg) throws IOException {
                StringBuilder builder = new StringBuilder();
 
                String startAt = "id=\"sfContentBody";
index f989663e24b0ae142f03448c1dc7d592b7204ba6..645d042e74467a3b8a2497f9b287fd1200b39a4b 100644 (file)
@@ -17,6 +17,7 @@ import be.nikiroo.fanfix.data.Story;
 import be.nikiroo.fanfix.supported.BasicSupport;
 import be.nikiroo.fanfix.supported.BasicSupport.SupportType;
 import be.nikiroo.utils.IOUtils;
+import be.nikiroo.utils.Progress;
 import be.nikiroo.utils.test.TestCase;
 import be.nikiroo.utils.test.TestLauncher;
 
@@ -60,13 +61,13 @@ public class BasicSupportTest extends TestLauncher {
 
                                                List<Paragraph> paras = null;
 
-                                               paras = support.makeParagraphs(null, "");
+                                               paras = support.makeParagraphs(null, "", null);
                                                assertEquals(
                                                                "An empty content should not generate paragraphs",
                                                                0, paras.size());
 
                                                paras = support.makeParagraphs(null,
-                                                               "Line 1</p><p>Line 2</p><p>Line 3</p>");
+                                                               "Line 1</p><p>Line 2</p><p>Line 3</p>", null);
                                                assertEquals(5, paras.size());
                                                assertEquals("Line 1", paras.get(0).getContent());
                                                assertEquals(ParagraphType.BLANK, paras.get(1)
@@ -77,7 +78,7 @@ public class BasicSupportTest extends TestLauncher {
                                                assertEquals("Line 3", paras.get(4).getContent());
 
                                                paras = support.makeParagraphs(null,
-                                                               "<p>Line1</p><p>Line2</p><p>Line3</p>");
+                                                               "<p>Line1</p><p>Line2</p><p>Line3</p>", null);
                                                assertEquals(6, paras.size());
                                        }
                                });
@@ -95,34 +96,39 @@ public class BasicSupportTest extends TestLauncher {
                                                List<Paragraph> paras = null;
 
                                                paras = support
-                                                               .makeParagraphs(null,
-                                                                               "<p>Line1</p><p>Line2</p><p>Line3<br/><br><p></p>");
+                                                               .makeParagraphs(
+                                                                               null,
+                                                                               "<p>Line1</p><p>Line2</p><p>Line3<br/><br><p></p>",
+                                                                               null);
                                                assertEquals(5, paras.size());
 
                                                paras = support
-                                                               .makeParagraphs(null,
-                                                                               "<p>Line1</p><p>Line2</p><p>Line3<br/><br><p></p>* * *");
+                                                               .makeParagraphs(
+                                                                               null,
+                                                                               "<p>Line1</p><p>Line2</p><p>Line3<br/><br><p></p>* * *",
+                                                                               null);
                                                assertEquals(5, paras.size());
 
-                                               paras = support.makeParagraphs(null, "1<p>* * *<p>2");
+                                               paras = support.makeParagraphs(null, "1<p>* * *<p>2",
+                                                               null);
                                                assertEquals(3, paras.size());
                                                assertEquals(ParagraphType.BREAK, paras.get(1)
                                                                .getType());
 
                                                paras = support.makeParagraphs(null,
-                                                               "1<p><br/><p>* * *<p>2");
+                                                               "1<p><br/><p>* * *<p>2", null);
                                                assertEquals(3, paras.size());
                                                assertEquals(ParagraphType.BREAK, paras.get(1)
                                                                .getType());
 
                                                paras = support.makeParagraphs(null,
-                                                               "1<p>* * *<br/><p><br><p>2");
+                                                               "1<p>* * *<br/><p><br><p>2", null);
                                                assertEquals(3, paras.size());
                                                assertEquals(ParagraphType.BREAK, paras.get(1)
                                                                .getType());
 
                                                paras = support.makeParagraphs(null,
-                                                               "1<p><br/><br>* * *<br/><p><br><p>2");
+                                                               "1<p><br/><br>* * *<br/><p><br><p>2", null);
                                                assertEquals(3, paras.size());
                                                assertEquals(ParagraphType.BREAK, paras.get(1)
                                                                .getType());
@@ -417,21 +423,21 @@ public class BasicSupportTest extends TestLauncher {
 
                @Override
                protected List<Entry<String, URL>> getChapters(URL source,
-                               InputStream in) throws IOException {
+                               InputStream in, Progress pg) throws IOException {
                        return null;
                }
 
                @Override
                protected String getChapterContent(URL source, InputStream in,
-                               int number) throws IOException {
+                               int number, Progress pg) throws IOException {
                        return null;
                }
 
                @Override
                // and make it public!
-               public List<Paragraph> makeParagraphs(URL source, String content)
-                               throws IOException {
-                       return super.makeParagraphs(source, content);
+               public List<Paragraph> makeParagraphs(URL source, String content,
+                               Progress pg) throws IOException {
+                       return super.makeParagraphs(source, content, pg);
                }
 
                @Override