Harmonize the find in page dialog.

chrome/browser/ui/views/harmony/harmony_layout_provider.cc

Index: chrome/browser/ui/views/harmony/harmony_layout_provider.cc
diff --git a/chrome/browser/ui/views/harmony/harmony_layout_provider.cc b/chrome/browser/ui/views/harmony/harmony_layout_provider.cc
index 29d289bb27dadc3d8a0abdfe127f059040f50a59..f87a869147baaa4f4f9b3d074b5c19e8a7368fa8 100644
--- a/chrome/browser/ui/views/harmony/harmony_layout_provider.cc
+++ b/chrome/browser/ui/views/harmony/harmony_layout_provider.cc
@@ -71,6 +71,12 @@ int HarmonyLayoutProvider::GetDistanceMetric(int metric) const {
       return kHarmonyLayoutUnit;
     case DISTANCE_UNRELATED_CONTROL_VERTICAL_LARGE:
       return kHarmonyLayoutUnit;
+    case DISTANCE_TOAST_BUTTON_VERTICAL:

[Peter Kasting, 2017/07/13 04:52:47]

I don't think you need to override any of these, since they all match your base
values.

[ananta, 2017/07/13 05:22:11]

Not doing this caused the NOTREACHED() in the function to fire for harmony

[Peter Kasting, 2017/07/13 05:31:37]

Hmm.  I'd say, remove the NOTREACHED and return 0, and instead add a default
case to the loop that calls the ChromeLayoutProvider, as we did with the insets
function above.

[ananta, 2017/07/13 05:43:49]

Thanks. Done

+      return kHarmonyLayoutUnit / 2;
+    case DISTANCE_TOAST_TEXT_VERTICAL:
+      return kHarmonyLayoutUnit / 2 ;
+    case DISTANCE_TOAST_LABEL_VERTICAL:
+      return 12;
   }
   NOTREACHED();
   return 0;