Re-use NoUnderlineClickableSpan which shows a clickable link with underlines turned off

chrome/android/java/src/org/chromium/chrome/browser/signin/AccountSigninView.java

 // Copyright 2015 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.
 
 package org.chromium.chrome.browser.signin;
 
 import android.app.FragmentManager;
 import android.content.Context;
 import android.graphics.Bitmap;
 import android.graphics.drawable.Drawable;
 import android.os.Bundle;
 import android.text.TextPaint;
 import android.text.TextUtils;
 import android.text.method.LinkMovementMethod;
-import android.text.style.ClickableSpan;
 import android.util.AttributeSet;
 import android.view.View;
 import android.widget.AdapterView;
 import android.widget.ArrayAdapter;
 import android.widget.Button;
 import android.widget.ImageView;
 import android.widget.LinearLayout;
 import android.widget.Spinner;
 import android.widget.TextView;
 
 import org.chromium.base.ApiCompatibilityUtils;
 import org.chromium.base.metrics.RecordUserAction;
 import org.chromium.chrome.R;
 import org.chromium.chrome.browser.firstrun.FirstRunView;
 import org.chromium.chrome.browser.firstrun.ImageCarousel;
 import org.chromium.chrome.browser.firstrun.ImageCarousel.ImageCarouselPositionChangeListener;
 import org.chromium.chrome.browser.firstrun.ProfileDataCache;
 import org.chromium.chrome.browser.preferences.PrefServiceBridge;
 import org.chromium.chrome.browser.profiles.ProfileDownloader;
 import org.chromium.chrome.browser.signin.AccountTrackerService.OnSystemAccountsSeededListener;
 import org.chromium.chrome.browser.sync.ui.ConfirmImportSyncDataDialog;
 import org.chromium.chrome.browser.sync.ui.ConfirmImportSyncDataDialog.ImportSyncType;
 import org.chromium.signin.InvestigatedScenario;
 import org.chromium.sync.signin.AccountManagerHelper;
+import org.chromium.ui.text.NoUnderlineClickableSpan;
 import org.chromium.ui.text.SpanApplier;
 import org.chromium.ui.text.SpanApplier.SpanInfo;
 
 import java.util.List;
 
 /**
  * This view allows the user to select an account to log in to, add an account,
  * cancel account selection, etc. Users of this class should
  * {@link AccountSigninView#setListener(Listener)} and
  * {@link AccountSigninView#setDelegate(Delegate)} after the view has been inflated.
@@ skipping 430 matching lines @@
 
     private void showConfirmSigninPage() {
         mSignedIn = true;
         updateProfileName();
 
         configureSpinner(false);
         setUpConfirmButton();
         setUpUndoButton();
 
         if (mShowSettingsSpan) {
-            ClickableSpan settingsSpan = new ClickableSpan() {
+            NoUnderlineClickableSpan settingsSpan = new NoUnderlineClickableSpan() {
                 @Override
                 public void onClick(View widget) {
                     mListener.onAccountSelected(mAccountName, true);
                 }
 
                 @Override
                 public void updateDrawState(TextPaint textPaint) {

[newt (away), 2016/03/10 02:45:14]

Remove this method. Again, we shouldn't need to override the link color here.

[juncai, 2016/03/11 23:53:05]

Done.

+                    super.updateDrawState(textPaint);
                     textPaint.setColor(getResources().getColor(R.color.ui_link_text_color));
-                    textPaint.setUnderlineText(false);
                 }
             };
             mDescriptionText.setText(SpanApplier.applySpans(getSignedInDescription(mIsChildAccount),
                     new SpanInfo(SETTINGS_LINK_OPEN, SETTINGS_LINK_CLOSE, settingsSpan)));
         } else {
             // If we aren't showing the span, get rid of the LINK1 annotations.
             mDescriptionText.setText(getSignedInDescription(mIsChildAccount)
                                              .replace(SETTINGS_LINK_OPEN, "")
                                              .replace(SETTINGS_LINK_CLOSE, ""));
         }
@@ skipping 151 matching lines @@
     public boolean isInForcedAccountMode() {
         return mForcedAccountName != null;
     }
 
     @Override
     public void onPositionChanged(int i) {
         mPositionSetProgrammatically = true;
         mSpinner.setSelection(i);
     }
 }