Bug 1243558 - Add UrlAnnotations.insertScreenshot method. r=sebastian draft
authorMichael Comella <michael.l.comella@gmail.com>
Fri, 26 Feb 2016 18:24:41 -0800
changeset 335633 cb04a2892490fd23b48f8b14d3b4c662e2e7e8a7
parent 335144 c5e44d32490584f7308b7886b263e68e5b95979e
child 335634 7521f7d902e9a70adfb49d0b4dfa2be4917c9287
push id11839
push usermichael.l.comella@gmail.com
push dateTue, 01 Mar 2016 01:32:00 +0000
reviewerssebastian
bugs1243558
milestone47.0a1
Bug 1243558 - Add UrlAnnotations.insertScreenshot method. r=sebastian MozReview-Commit-ID: KP37hhOxatN
mobile/android/base/java/org/mozilla/gecko/db/LocalUrlAnnotations.java
mobile/android/base/java/org/mozilla/gecko/db/StubBrowserDB.java
mobile/android/base/java/org/mozilla/gecko/db/UrlAnnotations.java
--- a/mobile/android/base/java/org/mozilla/gecko/db/LocalUrlAnnotations.java
+++ b/mobile/android/base/java/org/mozilla/gecko/db/LocalUrlAnnotations.java
@@ -5,17 +5,16 @@
 package org.mozilla.gecko.db;
 
 import android.content.ContentResolver;
 import android.content.ContentValues;
 import android.database.Cursor;
 import android.net.Uri;
 import android.support.annotation.NonNull;
 import android.support.annotation.Nullable;
-import org.json.JSONObject;
 import org.mozilla.gecko.annotation.RobocopTarget;
 import org.mozilla.gecko.db.BrowserContract.UrlAnnotations.Key;
 
 public class LocalUrlAnnotations implements UrlAnnotations {
     private Uri urlAnnotationsTableWithProfile;
 
     public LocalUrlAnnotations(final String profile) {
         urlAnnotationsTableWithProfile = DBUtils.appendProfile(profile, BrowserContract.UrlAnnotations.CONTENT_URI);
@@ -50,9 +49,13 @@ public class LocalUrlAnnotations impleme
                         BrowserContract.UrlAnnotations._ID,
                         BrowserContract.UrlAnnotations.URL,
                         BrowserContract.UrlAnnotations.KEY,
                         BrowserContract.UrlAnnotations.VALUE,
                         BrowserContract.UrlAnnotations.DATE_CREATED,
                 },
                 BrowserContract.UrlAnnotations.DATE_CREATED + " DESC");
     }
+
+    public void insertScreenshot(final ContentResolver cr, final String pageUrl, final String screenshotPath) {
+        insertAnnotation(cr, pageUrl, Key.SCREENSHOT.getDbValue(), screenshotPath);
+    }
 }
--- a/mobile/android/base/java/org/mozilla/gecko/db/StubBrowserDB.java
+++ b/mobile/android/base/java/org/mozilla/gecko/db/StubBrowserDB.java
@@ -151,16 +151,19 @@ class StubTabsAccessor implements TabsAc
 }
 
 class StubUrlAnnotations implements UrlAnnotations {
     @Override
     public void insertAnnotation(ContentResolver cr, String url, String key, String value) {}
 
     @Override
     public Cursor getScreenshots(ContentResolver cr) { return null; }
+
+    @Override
+    public void insertScreenshot(ContentResolver cr, String pageUrl, final String screenshotLocation) {}
 }
 
 /*
  * This base implementation just stubs all methods. For the
  * real implementations, see LocalBrowserDB.java.
  */
 public class StubBrowserDB implements BrowserDB {
     private final StubSearches searches = new StubSearches();
--- a/mobile/android/base/java/org/mozilla/gecko/db/UrlAnnotations.java
+++ b/mobile/android/base/java/org/mozilla/gecko/db/UrlAnnotations.java
@@ -5,10 +5,12 @@
 package org.mozilla.gecko.db;
 
 import android.content.ContentResolver;
 import android.database.Cursor;
 import org.mozilla.gecko.annotation.RobocopTarget;
 
 public interface UrlAnnotations {
     @RobocopTarget void insertAnnotation(ContentResolver cr, String url, String key, String value);
+
     Cursor getScreenshots(ContentResolver cr);
+    void insertScreenshot(ContentResolver cr, String pageUrl, String screenshotPath);
 }