Print Preview cleanup: fix indentation in destination_list_item.html

R=dpapad@chromium.org

Bug: None
Change-Id: I6e20db96c8c8a831ff6c624765a6a078a06e22fe
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2578704
Commit-Queue: dpapad <dpapad@chromium.org>
Auto-Submit: Rebekah Potter <rbpotter@chromium.org>
Reviewed-by: dpapad <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#835143}

chrome/browser/resources/print_preview/ui/destination_list_item.html

+<style include="cr-hidden-style">
+  :host {
+    align-items: center;
+    cursor: default;
+    display: flex;
+    font-size: calc(12/13 * 1em);
+    min-height: var(--destination-item-height);
+    opacity: .87;
+    padding-inline-end: 2px;
+    padding-inline-start: 14px;
+    vertical-align: middle;
+  }
 
-    <style include="cr-hidden-style">
-      :host {
-        align-items: center;
-        cursor: default;
-        display: flex;
-        font-size: calc(12/13 * 1em);
-        min-height: var(--destination-item-height);
-        opacity: .87;
-        padding-inline-end: 2px;
-        padding-inline-start: 14px;
-        vertical-align: middle;
-      }
+  :host > * {
+    align-items: center;
+    color: var(--cr-secondary-text-color);
+    font-size: calc(10/12 * 1em);
+    margin-inline-start: 1em;
+    overflow: hidden;
+    text-overflow: ellipsis;
+    vertical-align: middle;
+    white-space: nowrap;
+  }
 
-      :host > * {
-        align-items: center;
-        color: var(--cr-secondary-text-color);
-        font-size: calc(10/12 * 1em);
-        margin-inline-start: 1em;
-        overflow: hidden;
-        text-overflow: ellipsis;
-        vertical-align: middle;
-        white-space: nowrap;
-      }
+  iron-icon {
+    fill: var(--google-grey-600);
+    flex: 0;
+    height: 20px;
+    margin-inline-end: 12px;
+    min-width: 20px;
+    transition: opacity 150ms;
+  }
 
-      iron-icon {
-        fill: var(--google-grey-600);
-        flex: 0;
-        height: 20px;
-        margin-inline-end: 12px;
-        min-width: 20px;
-        transition: opacity 150ms;
-      }
+  @media (prefers-color-scheme: dark) {
+    iron-icon {
+      fill: var(--google-grey-refresh-500);
+    }
+  }
 
-      @media (prefers-color-scheme: dark) {
-        iron-icon {
-          fill: var(--google-grey-refresh-500);
-        }
-      }
+  .name {
+    color: var(--cr-primary-text-color);
+    font-size: 1em;
+    margin-inline-start: 0;
+  }
 
-      .name {
-        color: var(--cr-primary-text-color);
-        font-size: 1em;
-        margin-inline-start: 0;
-      }
+  .extension-controlled-indicator {
+    display: flex;
+    flex: 1;
+    justify-content: flex-end;
+    min-width: 150px;
+  }
 
-      .extension-controlled-indicator {
-        display: flex;
-        flex: 1;
-        justify-content: flex-end;
-        min-width: 150px;
-      }
+  .extension-icon {
+    height: 24px;
+    margin-inline-start: 1em;
+    width: 24px;
+  }
 
-      .extension-icon {
-        height: 24px;
-        margin-inline-start: 1em;
-        width: 24px;
-      }
+  .configuring-failed-text {
+    color: var(--google-red-600);
+    font-style: italic;
+  }
 
-      .configuring-failed-text {
-        color: var(--google-red-600);
-        font-style: italic;
-      }
-
-      :host([stale_]) :-webkit-any(iron-icon, .name, .connection-status) {
-        opacity: 0.4;
-      }
+  :host([stale_]) :-webkit-any(iron-icon, .name, .connection-status) {
+    opacity: 0.4;
+  }
 
 <if expr="chromeos">
-      :host([is-destination-cros-local_]) .connection-status {
-        color: var(--google-red-600);
-      }
+  :host([is-destination-cros-local_]) .connection-status {
+    color: var(--google-red-600);
+  }
 </if>
-    </style>
-    <iron-icon icon="[[destinationIcon_]]"></iron-icon>
-    <span class="name searchable">[[destination.displayName]]</span>
-    <span class="search-hint searchable" hidden="[[!searchHint_]]">
-      [[searchHint_]]
-    </span>
-    <span class="connection-status" hidden="[[!statusText_]]">
-      [[statusText_]]
-    </span>
-    <a href="$i18n{gcpCertificateErrorLearnMoreURL}"
-        target="_blank" class="learn-more-link"
-        hidden$="[[!destination.shouldShowInvalidCertificateError]]">
-      $i18n{learnMore}
-    </a>
-    <span class="extension-controlled-indicator"
-        hidden$="[[!destination.isExtension]]">
-      <span class="extension-name searchable">
-        [[destination.extensionName]]
-      </span>
-      <span class="extension-icon" role="button" tabindex="0"
-          title="[[getExtensionPrinterTooltip_(destination)]]"></span>
-    </span>
+</style>
+<iron-icon icon="[[destinationIcon_]]"></iron-icon>
+<span class="name searchable">[[destination.displayName]]</span>
+<span class="search-hint searchable" hidden="[[!searchHint_]]">
+  [[searchHint_]]
+</span>
+<span class="connection-status" hidden="[[!statusText_]]">
+  [[statusText_]]
+</span>
+<a href="$i18n{gcpCertificateErrorLearnMoreURL}"
+    target="_blank" class="learn-more-link"
+    hidden$="[[!destination.shouldShowInvalidCertificateError]]">
+  $i18n{learnMore}
+</a>
+<span class="extension-controlled-indicator"
+    hidden$="[[!destination.isExtension]]">
+  <span class="extension-name searchable">
+    [[destination.extensionName]]
+  </span>
+  <span class="extension-icon" role="button" tabindex="0"
+      title="[[getExtensionPrinterTooltip_(destination)]]"></span>
+</span>
 <if expr="chromeos">
-    <span class="configuring-in-progress-text"
-      hidden$="[[!checkConfigurationStatus_(statusEnum_.IN_PROGRESS,
-                                            configurationStatus_)]]">
-      $i18n{configuringInProgressText}
-      <span class="configuring-text-jumping-dots">
-        <span>.</span><span>.</span><span>.</span>
-      </span>
-    </span>
-    <span class="configuring-failed-text"
-      hidden$="[[!checkConfigurationStatus_(statusEnum_.FAILED,
-                                            configurationStatus_)]]">
-      $i18n{configuringFailedText}
-    </span>
+<span class="configuring-in-progress-text"
+  hidden$="[[!checkConfigurationStatus_(statusEnum_.IN_PROGRESS,
+                                        configurationStatus_)]]">
+  $i18n{configuringInProgressText}
+  <span class="configuring-text-jumping-dots">
+    <span>.</span><span>.</span><span>.</span>
+  </span>
+</span>
+<span class="configuring-failed-text"
+  hidden$="[[!checkConfigurationStatus_(statusEnum_.FAILED,
+                                        configurationStatus_)]]">
+  $i18n{configuringFailedText}
+</span>
 </if>

chrome/test/data/webui/print_preview/destination_item_test.js

@@ -139,9 +139,8 @@ suite(destination_item_test.suiteName, function() {
 
     // Search hint should be have the description and be highlighted.
     const hint = item.$$('.search-hint');
-    assertEquals(
-        params.description + '\n    \n      ' + params.description,
-        hint.textContent.trim());
+    assertTrue(hint.textContent.includes(params.description));
+    assertFalse(hint.textContent.includes(params.location));
     const searchHits = hint.querySelectorAll('.search-highlight-hit');
     assertEquals(1, searchHits.length);
     assertEquals('ABC', searchHits[0].textContent);