Fix Bookmarks bubble title misalignment.

chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc

Index: chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc
diff --git a/chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc b/chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc
index 0e8c261e3a902436d472a5d049443203c147a18e..2d3be5db1be69acba1fd69e479a8a401112310bb 100644
--- a/chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc
+++ b/chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc
@@ -303,7 +303,14 @@ BookmarkBubbleView::BookmarkBubbleView(
       ios_promo_view_(nullptr),
       remove_bookmark_(false),
       apply_edits_(true),
-      is_showing_ios_promotion_(false) {}
+      is_showing_ios_promotion_(false) {
+  // Set title margins to make the title and the content left aligned.
+  const int side_margin = margins().left();
+  set_title_margins(
+      gfx::Insets(LayoutDelegate::Get()->GetMetric(
+                      LayoutDelegate::Metric::PANEL_CONTENT_MARGIN),
+                  side_margin, 0, side_margin));

[Peter Kasting, 2017/04/11 20:26:29]

Are the title and content misaligned on other similar dialogs?  If so, this fix
doesn't seem correct because it only adjusts bookmarks.  If not, why are other
places correct but this one isn't?

[mrefaat1, 2017/04/11 20:44:02]

i'm not sure if its a problem on different dialogs, But it was broken on
Passwords dialog and someone also sent a fix that does the same thing (only on
the passwords) i'd definite prefer to have a fix for all bubbles - But if this
is going to be fixed with harmony is it worth it to do a fix ? 

[Peter Kasting, 2017/04/11 20:57:05]

Where is this passwords-specific fix?  Did it land?  If so let's back it out and
fix in a more global fashion.

We should definitely fix this stuff now, pre-Harmony, because Harmony assumes
the existing behavior is correct and tries to refactor and modify code based on
that.  If the existing behavior is wrong, we're going to be really confused.

[mrefaat, 2017/04/12 18:43:23]

this is the specific fix:
https://chromium.googlesource.com/chromium/src.git/+/623dc2bca68212e0f7e4617e...
- it did land and was merged to 58 too.

+}
 
 base::string16 BookmarkBubbleView::GetTitle() {
   BookmarkModel* bookmark_model =