Decouple publishing from sharing in PublishPageCallback.

Bug: 758690
Change-Id: I829790f7f53d97b4717b653b3eeae9065387e7ad
Reviewed-on: https://chromium-review.googlesource.com/994336Reviewed-by: Jian Li <jianli@chromium.org>
Commit-Queue: Peter Williamson <petewil@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547906}

chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageUtils.java

@@ -458,8 +458,15 @@ public class OfflinePageUtils {
     }
 
     /**
-     * Callback from the native code to publish a page.  This will take the page, now in
-     * a public directory, and share it.
+     * Called when publishing is done.  Continues with processing to share.
+     */
+    public static void publishCompleted(OfflinePageItem page, final Activity activity,
+            final Callback<ShareParams> shareCallback) {
+        sharePublishedPage(page, activity, shareCallback);
+    }
+
+    /**
+     * This will take a page in a public directory, and share it.
      */
     public static void sharePublishedPage(OfflinePageItem page, final Activity activity,
             final Callback<ShareParams> shareCallback) {

chrome/android/java/src/org/chromium/chrome/browser/offlinepages/PublishPageCallback.java

@@ -45,6 +45,6 @@ public class PublishPageCallback implements Callback<String> {
 
         // TODO(petewil): Sharing seems out of place here. Move the call to sharing
         // back to OfflinePageUtils.
-        OfflinePageUtils.sharePublishedPage(page, mActivity, mShareCallback);
+        OfflinePageUtils.publishCompleted(page, mActivity, mShareCallback);
     }
 }