[Offline pages] Refactor URL conversions from TabAndroid

components/offline_pages/offline_page_test_archiver.h

Index: components/offline_pages/offline_page_test_archiver.h
diff --git a/components/offline_pages/offline_page_test_archiver.h b/components/offline_pages/offline_page_test_archiver.h
index 7375c1868d23cf497f1984eab9eae73946fb6a0a..a8a93830a8d27d3ab862f55482f38808125452ee 100644
--- a/components/offline_pages/offline_page_test_archiver.h
+++ b/components/offline_pages/offline_page_test_archiver.h
@@ -22,6 +22,8 @@ namespace offline_pages {
 // for an actual web contents.
 class OfflinePageTestArchiver : public OfflinePageArchiver {
  public:
+  // TODO(fgorski): Try refactoring the observer out and replace it with a
+  // callback, or completely remove the call to |SetLastPathCreatedByArchiver|.
   class Observer {
    public:
     virtual ~Observer() {}
@@ -47,8 +49,13 @@ class OfflinePageTestArchiver : public OfflinePageArchiver {
 
   // When set to true, |CompleteCreateArchive| should be called explicitly for
   // the process to finish.
+  // TODO(fgorski): See if we can move this to the constructor.
   void set_delayed(bool delayed) { delayed_ = delayed; }
 
+  // Allows to explicitly specify a file name for the tests.
+  // TODO(fgorski): See if we can move this to the constructor.
+  void set_filename(const base::FilePath& filename) { filename_ = filename; }
+
   bool create_archive_called() const { return create_archive_called_; }
 
  private:
@@ -56,6 +63,7 @@ class OfflinePageTestArchiver : public OfflinePageArchiver {
   Observer* observer_;
   GURL url_;
   base::FilePath archives_dir_;
+  base::FilePath filename_;
   ArchiverResult result_;
   int64 size_to_report_;
   bool create_archive_called_;