MD History: Refactored design for displaying history information

chrome/browser/resources/md_history/history_item.js

Index: chrome/browser/resources/md_history/history_item.js
diff --git a/chrome/browser/resources/md_history/history_item.js b/chrome/browser/resources/md_history/history_item.js
index 242660db8aaf7a02a7c50c78b4db2e6fa31360de..47238cc7bee77edf64bb3e540d5bd146e6efbd68 100644
--- a/chrome/browser/resources/md_history/history_item.js
+++ b/chrome/browser/resources/md_history/history_item.js
@@ -6,12 +6,18 @@ Polymer({
   is: 'history-item',
 
   properties: {
-    timeAccessed_: {
+    // The date of these history items.
+    historyDate: {
       type: String,
       value: ''
     },
 
-    websiteTitle_: {
+    timeAccessed: {

[calamity, 2016/02/09 12:25:29]

I recommend you change this to cardTitle in anticipation of hsampson's change. I
believe she's going to using this field for the search result card title.

[yingran, 2016/02/11 00:41:42]

??? do u mean historyDate -> cardTitle? timeAccessed is still in her search
thing...

[tsergeant, 2016/02/11 02:43:02]

I think that this will no longer need to change, yeah. Hannah and I went back on
that and historyDate should still exist.

[yingran, 2016/02/12 02:03:05]

coolies

+      type: String,
+      value: ''
+    },
+
+    websiteTitle: {
       type: String,
       value: ''
     },
@@ -20,7 +26,7 @@ Polymer({
     // Gives the user some idea of which history items are different pages
     // belonging to the same site, and can be used to look for more items
     // from the same site.
-    websiteDomain_: {
+    websiteDomain: {
       type: String,
       value: ''
     },
@@ -28,7 +34,7 @@ Polymer({
     // The website url is used to define where the link should take you if
     // you click on the title, and also to define which icon the history-item
     // should display.
-    websiteUrl_: {
+    websiteUrl: {
       type: String,
       value: '',
       observer: 'showIcon_'
@@ -42,7 +48,7 @@ Polymer({
     },
 
     // The time in seconds of when the website was accessed.
-    timestamp_: {
+    timestamp: {
       type: Number,
       value: 0
     },
@@ -51,6 +57,23 @@ Polymer({
       type: Boolean,
       value: false,
       notify: true
+    },
+
+    isCardStart: {
+      type: Boolean,
+      value: false,
+      reflectToAttribute: true
+    },
+
+    isCardEnd: {
+      type: Boolean,
+      value: false,
+      reflectToAttribute: true
+    },
+
+    hasTimeGap: {
+      type: Boolean,
+      value: false
     }
   },
 
@@ -72,7 +95,7 @@ Polymer({
    */
   showIcon_: function() {
     this.$['website-icon'].style.backgroundImage =
-        getFaviconImageSet(this.websiteUrl_);
+        getFaviconImageSet(this.websiteUrl);
   },
 
   /**
@@ -85,7 +108,7 @@ Polymer({
     this.fire('toggle-menu', {
       x: position.left,
       y: position.top,
-      accessTime: this.timestamp_
+      accessTime: this.timestamp
     });
 
     // Stops the 'tap' event from closing the menu when it opens.