OfflineContentProvider changes to start service

components/offline_items_collection/core/android/java/src/org/chromium/components/offline_items_collection/OfflineItem.java

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.components.offline_items_collection;
 
 import org.chromium.base.annotations.SuppressFBWarnings;
 import org.chromium.components.offline_items_collection.OfflineItemFilter.OfflineItemFilterEnum;
 import org.chromium.components.offline_items_collection.OfflineItemState.OfflineItemStateEnum;
 
@@ skipping 26 matching lines @@
     public boolean isOffTheRecord;
 
     // In Progress Metadata.
     @OfflineItemStateEnum
     public int state;
     public boolean isResumable;
     public long receivedBytes;
     public int percentCompleted;
     public long timeRemainingMs;
 
-    OfflineItem() {
-        id = new ContentId();
+    public OfflineItem() {
+        this.id = new ContentId();

[gone, 2017/03/20 19:03:36]

why add the this. for one of them but not the other two?

[David Trainor- moved to gerrit, 2017/03/25 03:31:13]

Ah sorry I used to allow passing in an id, so I had "this.id = id;"  Didn't
remove the this.

         filter = OfflineItemFilter.FILTER_OTHER;
         state = OfflineItemState.COMPLETE;

[qinmin, 2017/03/22 07:53:58]

should the initial state is COMPLETE?

     }
+
+    @Override
+    public String toString() {
+        return getClass().getSimpleName() + "@" + Integer.toHexString(hashCode()) + " [" + id + "]";

[gone, 2017/03/20 19:03:36]

was this thing automatically added?

[David Trainor- moved to gerrit, 2017/03/25 03:31:13]

No but it makes debugging unit tests easier as mockito likes to print out the
object when things don't work as expected.

+    }
 }