Commit 84f3ec82 authored by Shakti Sahu's avatar Shakti Sahu Committed by Commit Bot

Added content quality score to OfflineItem

Change-Id: I7d24d5b4cd1f45c1ae1c268b14c6f1c779caf2fb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1866012
Commit-Queue: Shakti Sahu <shaktisahu@chromium.org>
Reviewed-by: default avatarDavid Trainor <dtrainor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#706905}
parent 7669c483
...@@ -77,6 +77,7 @@ public class OfflineItem implements Cloneable { ...@@ -77,6 +77,7 @@ public class OfflineItem implements Cloneable {
public boolean promoteOrigin; public boolean promoteOrigin;
public boolean canRename; public boolean canRename;
public boolean ignoreVisuals; public boolean ignoreVisuals;
public double contentQualityScore;
// Content Metadata. // Content Metadata.
public long totalSizeBytes; public long totalSizeBytes;
...@@ -134,6 +135,7 @@ public class OfflineItem implements Cloneable { ...@@ -134,6 +135,7 @@ public class OfflineItem implements Cloneable {
clone.mimeType = mimeType; clone.mimeType = mimeType;
clone.canRename = canRename; clone.canRename = canRename;
clone.ignoreVisuals = ignoreVisuals; clone.ignoreVisuals = ignoreVisuals;
clone.contentQualityScore = contentQualityScore;
clone.pageUrl = pageUrl; clone.pageUrl = pageUrl;
clone.originalUrl = originalUrl; clone.originalUrl = originalUrl;
clone.isOffTheRecord = isOffTheRecord; clone.isOffTheRecord = isOffTheRecord;
......
...@@ -54,7 +54,7 @@ public final class OfflineItemBridge { ...@@ -54,7 +54,7 @@ public final class OfflineItemBridge {
@FailState int failState, @PendingState int pendingState, boolean isResumable, @FailState int failState, @PendingState int pendingState, boolean isResumable,
boolean allowMetered, long receivedBytes, long progressValue, long progressMax, boolean allowMetered, long receivedBytes, long progressValue, long progressMax,
@OfflineItemProgressUnit int progressUnit, long timeRemainingMs, boolean isDangerous, @OfflineItemProgressUnit int progressUnit, long timeRemainingMs, boolean isDangerous,
boolean canRename, boolean ignoreVisuals) { boolean canRename, boolean ignoreVisuals, double contentQualityScore) {
OfflineItem item = new OfflineItem(); OfflineItem item = new OfflineItem();
item.id.namespace = nameSpace; item.id.namespace = nameSpace;
item.id.id = id; item.id.id = id;
...@@ -88,6 +88,7 @@ public final class OfflineItemBridge { ...@@ -88,6 +88,7 @@ public final class OfflineItemBridge {
item.isDangerous = isDangerous; item.isDangerous = isDangerous;
item.canRename = canRename; item.canRename = canRename;
item.ignoreVisuals = ignoreVisuals; item.ignoreVisuals = ignoreVisuals;
item.contentQualityScore = contentQualityScore;
if (list != null) list.add(item); if (list != null) list.add(item);
return item; return item;
} }
......
...@@ -45,7 +45,8 @@ JNI_OfflineItemBridge_createOfflineItemAndMaybeAddToList( ...@@ -45,7 +45,8 @@ JNI_OfflineItemBridge_createOfflineItemAndMaybeAddToList(
item.is_resumable, item.allow_metered, item.received_bytes, item.is_resumable, item.allow_metered, item.received_bytes,
item.progress.value, item.progress.max.value_or(-1), item.progress.value, item.progress.max.value_or(-1),
static_cast<jint>(item.progress.unit), item.time_remaining_ms, static_cast<jint>(item.progress.unit), item.time_remaining_ms,
item.is_dangerous, item.can_rename, item.ignore_visuals); item.is_dangerous, item.can_rename, item.ignore_visuals,
item.content_quality_score);
} }
} // namespace } // namespace
......
...@@ -46,6 +46,7 @@ OfflineItem::OfflineItem() ...@@ -46,6 +46,7 @@ OfflineItem::OfflineItem()
promote_origin(false), promote_origin(false),
can_rename(false), can_rename(false),
ignore_visuals(false), ignore_visuals(false),
content_quality_score(0),
total_size_bytes(0), total_size_bytes(0),
externally_removed(false), externally_removed(false),
is_openable(false), is_openable(false),
...@@ -77,6 +78,7 @@ bool OfflineItem::operator==(const OfflineItem& offline_item) const { ...@@ -77,6 +78,7 @@ bool OfflineItem::operator==(const OfflineItem& offline_item) const {
promote_origin == offline_item.promote_origin && promote_origin == offline_item.promote_origin &&
can_rename == offline_item.can_rename && can_rename == offline_item.can_rename &&
ignore_visuals == offline_item.ignore_visuals && ignore_visuals == offline_item.ignore_visuals &&
content_quality_score == offline_item.content_quality_score &&
total_size_bytes == offline_item.total_size_bytes && total_size_bytes == offline_item.total_size_bytes &&
externally_removed == offline_item.externally_removed && externally_removed == offline_item.externally_removed &&
creation_time == offline_item.creation_time && creation_time == offline_item.creation_time &&
......
...@@ -128,6 +128,11 @@ struct OfflineItem { ...@@ -128,6 +128,11 @@ struct OfflineItem {
// information. // information.
bool ignore_visuals; bool ignore_visuals;
// A score in the range of 0 to 1.0 showing how relevant the content is for
// the user. Higher score is an indication that the item will be ranked higher
// in the UI surfaces.
double content_quality_score;
// TODO(dtrainor): Build out custom per-item icon support. // TODO(dtrainor): Build out custom per-item icon support.
// Content Metadata. // Content Metadata.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment