add 'langauge settings' link to the translate bubble advanced view

chrome/browser/ui/cocoa/translate/translate_bubble_controller.mm

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "chrome/browser/ui/cocoa/translate/translate_bubble_controller.h"
 
 #include <utility>
 
 #include "base/mac/foundation_util.h"
 #include "base/mac/scoped_nsobject.h"
 #include "base/macros.h"
 #include "base/strings/sys_string_conversions.h"
+#include "chrome/browser/ui/chrome_pages.h"
 #import "chrome/browser/ui/cocoa/browser_window_controller.h"
 #import "chrome/browser/ui/cocoa/bubble_combobox.h"
 #import "chrome/browser/ui/cocoa/info_bubble_view.h"
 #import "chrome/browser/ui/cocoa/info_bubble_window.h"
 #import "chrome/browser/ui/cocoa/toolbar/toolbar_controller.h"
 #include "chrome/browser/ui/translate/language_combobox_model.h"
-#include "chrome/browser/ui/translate/translate_bubble_model_impl.h"
+#include "chrome/browser/ui/translate/translate_bubble_model.h"
+#include "chrome/browser/ui/translate/translate_bubble_view_state_transition.h"
+#include "chrome/common/url_constants.h"
 #include "chrome/grit/generated_resources.h"
 #include "components/translate/core/browser/translate_ui_delegate.h"
 #include "content/public/browser/browser_context.h"
+#include "content/public/common/referrer.h"
 #include "grit/components_strings.h"
 #include "ui/base/cocoa/cocoa_base_utils.h"
 #import "ui/base/cocoa/controls/hyperlink_button_cell.h"
 #import "ui/base/cocoa/window_size_constants.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/models/combobox_model.h"
 
 // TODO(hajimehoshi): This class is almost same as that of views. Refactor them.
 class TranslateDenialComboboxModel : public ui::ComboboxModel {
  public:
@@ skipping 60 matching lines @@
 - (NSButton*)addCheckbox:(NSString*)title
                   toView:(NSView*)view;
 - (NSPopUpButton*)addPopUpButton:(ui::ComboboxModel*)model
                           action:(SEL)action
                           toView:(NSView*)view;
 - (void)handleNopeButtonPressed;
 - (void)handleDoneButtonPressed;
 - (void)handleCancelButtonPressed;
 - (void)handleShowOriginalButtonPressed;
 - (void)handleAdvancedLinkButtonPressed;
+- (void)handleLanguageSettingsLinkButtonPressed;
 - (void)handleDenialPopUpButtonNopeSelected;
 - (void)handleDenialPopUpButtonNeverTranslateLanguageSelected;
 - (void)handleDenialPopUpButtonNeverTranslateSiteSelected;
 - (void)handleSourceLanguagePopUpButtonSelectedItemChanged:(id)sender;
 - (void)handleTargetLanguagePopUpButtonSelectedItemChanged:(id)sender;
 
 @end
 
 @implementation TranslateBubbleController
 
@@ skipping 320 matching lines @@
   // Buttons
   advancedDoneButton_ =
       [self addButton:l10n_util::GetNSStringWithFixup(IDS_DONE)
                action:@selector(handleDoneButtonPressed)
                toView:view];
   advancedCancelButton_ =
       [self addButton:l10n_util::GetNSStringWithFixup(IDS_CANCEL)
                action:@selector(handleCancelButtonPressed)
                toView:view];
 
+  NSString* message = l10n_util::GetNSStringWithFixup(
+      IDS_TRANSLATE_BUBBLE_LANGUAGE_SETTINGS);
+  action = @selector(handleLanguageSettingsLinkButtonPressed);
+  NSButton* advancedLanguageSettingsLinkButton =

[groby-ooo-7-16, 2016/05/18 23:08:16]

Why is this a button? Should this be a hyperlink?

[ftang, 2016/05/18 23:57:42]

I just reuse the pre-existing addLinkButtonWithText which show the "options"
link in these view. What Cocoa class do you have in mind for this instead?

[groby-ooo-7-16, 2016/05/19 01:38:09]

I was thinking of HyperLinkTextView - but addLinkButton seems to be a valid way
as well...

+      [self addLinkButtonWithText:message
+                           action:action
+                           toView:view];
+
   // Layout
   CGFloat textLabelWidth = NSWidth([sourceLanguageLabel frame]);
   if (textLabelWidth < NSWidth([targetLanguageLabel frame]))
     textLabelWidth = NSWidth([targetLanguageLabel frame]);
 
   CGFloat yPos = 0;
 
   [advancedDoneButton_ setFrameOrigin:NSMakePoint(0, yPos)];
   [advancedCancelButton_ setFrameOrigin:NSMakePoint(0, yPos)];
 
-  yPos += NSHeight([advancedDoneButton_ frame]) +
+  [advancedLanguageSettingsLinkButton setFrameOrigin:NSMakePoint(0, yPos)];
+
+  // Vertical center the advancedLanguageSettingsLinkButton with the
+  // advancedDoneButton_.
+  yPos = (NSHeight([advancedDoneButton_ frame]) -

[groby-ooo-7-16, 2016/05/19 01:38:09]

You don't want to center this with the button - you instead want to align the
baselines of the text in the buttons and the link.

Eyeballing the screenshot, you want to nudge the link 1pt up. (Please leave a
comment you're adjusting to align text baselines)

+         NSHeight([advancedLanguageSettingsLinkButton frame])) / 2;
+
+  [advancedLanguageSettingsLinkButton setFrameOrigin:NSMakePoint(0, yPos)];
+
+  yPos = NSHeight([advancedDoneButton_ frame]) +
       kUnrelatedControlVerticalSpacing;
 
   if (alwaysTranslateCheckbox_) {
     [alwaysTranslateCheckbox_ setFrameOrigin:NSMakePoint(textLabelWidth, yPos)];
 
     yPos += NSHeight([alwaysTranslateCheckbox_ frame]) +
         kRelatedControlVerticalSpacing;
   }
 
   CGFloat diffY = [[sourcePopUpButton cell]
@@ skipping 163 matching lines @@
 
 - (void)handleShowOriginalButtonPressed {
   model_->RevertTranslation();
   [self close];
 }
 
 - (void)handleAdvancedLinkButtonPressed {
   [self switchView:TranslateBubbleModel::VIEW_STATE_ADVANCED];
 }
 
+- (void)handleLanguageSettingsLinkButtonPressed {
+  GURL url = chrome::GetSettingsUrl(chrome::kLanguageOptionsSubPage);
+  webContents_->OpenURL(
+      content::OpenURLParams(url, content::Referrer(), NEW_FOREGROUND_TAB,
+                             ui::PAGE_TRANSITION_LINK, false));
+  translate::ReportUiAction(translate::SETTINGS_LINK_CLICKED);
+  [self close];
+}
+
 - (void)handleDenialPopUpButtonNopeSelected {
   model_->DeclineTranslation();
   [self close];
 }
 
 - (void)handleDenialPopUpButtonNeverTranslateLanguageSelected {
   model_->DeclineTranslation();
   model_->SetNeverTranslateLanguage(true);
   [self close];
 }
@@ skipping 10 matching lines @@
   [self updateAdvancedView];
 }
 
 - (void)handleTargetLanguagePopUpButtonSelectedItemChanged:(id)sender {
   NSPopUpButton* button = base::mac::ObjCCastStrict<NSPopUpButton>(sender);
   model_->UpdateTargetLanguageIndex([button indexOfSelectedItem]);
   [self updateAdvancedView];
 }
 
 @end