OfflineContentProvider changes to start service

components/offline_items_collection/core/offline_item_state.h

 // 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.
 
 #ifndef COMPONENTS_OFFLINE_ITEMS_COLLECTION_CORE_OFFLINE_ITEM_STATE_H_
 #define COMPONENTS_OFFLINE_ITEMS_COLLECTION_CORE_OFFLINE_ITEM_STATE_H_
 
 namespace offline_items_collection {
 
 // A Java counterpart will be generated for this enum.
 // GENERATED_JAVA_ENUM_PACKAGE: org.chromium.components.offline_items_collection
 enum OfflineItemState {
   IN_PROGRESS = 0,
+  PENDING,

[David Trainor- moved to gerrit, 2017/03/18 00:45:41]

Trying to find the right way to expose these.  I'd rather not have a ton of
methods on the NotifierUi interface above.  You should be able to pull the type
of notification you need to show from the OfflineItem state without the need of
an additional 'action' parameter.

I think OfflinePages needs PENDING, and these two states make sense to me as
OfflineItemStates anyway.

   COMPLETE,
   CANCELLED,
   INTERRUPTED,
+  FAILED,
   PAUSED,  // TODO(dtrainor): Make sure exposing a PAUSED state does not impact
            // downloads resumption.
   MAX_DOWNLOAD_STATE,
 };
 
 }  // namespace offline_items_collection
 
 #endif  // COMPONENTS_OFFLINE_ITEMS_COLLECTION_CORE_OFFLINE_ITEM_STATE_H_