https://codereview.chromium.org/2556563005

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4aff4df5f338661a2b9c5449ebfe0c33d4311d75

commit 4aff4df5f338661a2b9c5449ebfe0c33d4311d75
Author: ellyjones <ellyjones@chromium.org>
Date: Thu Dec 08 19:18:27 2016

views: add harmony control label alignment

Harmony specifies leading alignment for these; the old UI style
used trailing alignment mostly, but was inconsistent. This change:

1) Adds HarmonyLayoutDelegate::GetControlLabelGridAlignment();
2) Changes CookieInfoView (used in the collected cookies dialog) to
   use GetControlLabelGridAlignment;
3) Changes the bookmark bubble from leading to trailing alignment

BUG= 671828 

Review-Url: https://codereview.chromium.org/2556563005
Cr-Commit-Position: refs/heads/master@{#437303}

[modify] https://crrev.com/4aff4df5f338661a2b9c5449ebfe0c33d4311d75/chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc
[modify] https://crrev.com/4aff4df5f338661a2b9c5449ebfe0c33d4311d75/chrome/browser/ui/views/cookie_info_view.cc
[modify] https://crrev.com/4aff4df5f338661a2b9c5449ebfe0c33d4311d75/chrome/browser/ui/views/harmony/harmony_layout_delegate.cc
[modify] https://crrev.com/4aff4df5f338661a2b9c5449ebfe0c33d4311d75/chrome/browser/ui/views/harmony/harmony_layout_delegate.h
[modify] https://crrev.com/4aff4df5f338661a2b9c5449ebfe0c33d4311d75/chrome/browser/ui/views/harmony/layout_delegate.cc
[modify] https://crrev.com/4aff4df5f338661a2b9c5449ebfe0c33d4311d75/ui/views/layout/layout_constants.h

Is this bug Fixed (and further work to use the framework added here is part of the individual "convert dialog XXX" bugs)?

ellyjones@ is on leave but I believe this issue is fixed.

Yep, this is fixed. Individual dialogs still do need to make use of it but the machinery is there.