[MD History] Add routing to the synced devices page.

chrome/browser/resources/md_history/side_bar.html

 <link rel="import" href="chrome://resources/html/polymer.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/app-layout/app-drawer/app-drawer.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-selector/iron-selector.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-item/paper-item.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/paper-item/paper-item-shared-styles.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/paper-listbox/paper-listbox.html">

[tsergeant, 2016/07/13 07:23:33]

Listbox is unused, paper-item-shared-styles might be too?

[calamity, 2016/07/14 06:20:57]

paper-item-shared-styles are used for --paper-font-subhead but that include can
be made more specific.

+<link rel="import" href="chrome://resources/polymer/v1_0/paper-ripple/paper-ripple.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-styles/color.html">
 <link rel="import" href="chrome://history/shared_style.html">
 
 <dom-module id="history-side-bar">
   <template>
     <style include="shared-style">
       :host {
         display: block;
         padding-top: 5px;
         width: var(--side-bar-width);
@@ skipping 17 matching lines @@
       div.separator {
         background-color: rgba(0, 0, 0, 0.08);
         height: 1px;
         margin: 8px 0;
       }
 
       #clear-browsing-data {
         text-transform: uppercase;
       }
 
-      paper-item {
+      iron-selector {
+        -webkit-user-select: none;
+        background-color: transparent;
+        color: #5a5a5a;
+      }
+
+      iron-selector a {

[tsergeant, 2016/07/13 07:23:33]

Replacing paper-item with this CSS removes the focus highlight from the menu
items.

[calamity, 2016/07/14 06:20:57]

Fixed.

+        @apply(--paper-font-subhead);
         -webkit-padding-start: 25px;
+        align-items: center;
+        box-sizing: border-box;
+        color: inherit;
         cursor: pointer;
+        display: flex;
         font-size: 14px;
         font-weight: 500;
+        min-height: 48px;
+        outline: none;
+        position: relative;
+        text-decoration: none;
       }
 
-      paper-item.iron-selected {
+      iron-selector a.iron-selected {
         color: var(--google-blue-500);
         font-weight: 500;
       }
 
       iron-selector {
         -webkit-user-select: none;
         color: #5a5a5a;
       }
     </style>
 
     <app-drawer id="drawer" tabindex="0" on-focus="onDrawerFocus_" swipe-open
         align="start">
       <div id="drawer-header">
         <h1>$i18n{title}</h1>
       </div>
       <iron-selector id="menu" selected="{{selectedPage}}"
-          selectable=".page-item" attr-for-selected="view-id"
+          selectable=".page-item" attr-for-selected="path"
+          fallback-selection="history"
           on-iron-activate="onSelectorActivate_">
-        <paper-item view-id="history" class="page-item">
-          $i18n{historyMenuItem}
-        </paper-item>
-        <paper-item view-id="synced-devices" class="page-item">
-          $i18n{openTabsMenuItem}
-        </paper-item>
+        <a href="/" class="page-item" path="history">
+          <div>$i18n{historyMenuItem}</div>
+          <paper-ripple></paper-ripple>

[tsergeant, 2016/07/13 07:23:33]

The ripples were intentionally deleted from the sidebar, no need for them to
come back =)

(and if you remove the <paper-ripple> maybe you can have just the <a> with no
inner <div>?)

[calamity, 2016/07/14 06:20:57]

Done.

+        </a>
+        <a href="/syncedTabs" class="page-item" path="syncedTabs">
+          <div>$i18n{openTabsMenuItem}</div>
+          <paper-ripple></paper-ripple>
+        </a>
         <div class="separator"></div>
-        <paper-item on-tap="onClearBrowsingDataTap_" id="clear-browsing-data">
-          $i18n{clearBrowsingData}
-        </paper-item>
+        <a href="chrome://settings/clearBrowserData" id="clear-browsing-data">

[tsergeant, 2016/07/13 07:23:33]

I changed this button to use a chrome.send call, rebase to bring that change
back.

[calamity, 2016/07/14 06:20:57]

Done.

+          <div>$i18n{clearBrowsingData}</div>
+          <paper-ripple></paper-ripple>
+        </a>
       </iron-selector>
     </app-drawer>
   </template>
   <script src="chrome://history/side_bar.js"></script>
 </dom-module>