MD Settings: ensure <a is="action-link"> have no :hover underline

MD Settings: ensure <a is="action-link"> have no :hover underline

tl;dr - the style loading order matters, action_link.css must come
before <style include="settings-shared">.

R=hcarmona@chromium.org
BUG=630067
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/2ff90f4a2f6f30b5e01f9bef16cbe27efce16b01
Cr-Commit-Position: refs/heads/master@{#406755}

[Dan Beam, 2016-07-21 01:59:37]

Description was changed from

==========
MD Settings: ensure <a is="action-link"> have no :hover underline

tl;dr - the style loading order matters, action_link.css must come
before <style include="settings-shared">.

R=hcarmona@chromium.org
BUG=630067
==========

to

==========
MD Settings: ensure <a is="action-link"> have no :hover underline

tl;dr - the style loading order matters, action_link.css must come
before <style include="settings-shared">.

R=hcarmona@chromium.org
BUG=630067
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Dan Beam, 2016-07-21 01:59:59]

https://codereview.chromium.org/2167043002/diff/1/chrome/browser/resources/se...
File
chrome/browser/resources/settings/passwords_and_forms_page/passwords_section.html
(right):

https://codereview.chromium.org/2167043002/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/passwords_and_forms_page/passwords_section.html:54:
text-decoration: underline;
i'm going to change this separately

[Dan Beam, 2016-07-21 02:00:05]

The CQ bit was checked by dbeam@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-21 02:00:16]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[hcarmona, 2016-07-21 02:10:04]

LGTM

https://codereview.chromium.org/2167043002/diff/1/chrome/browser/resources/se...
File
chrome/browser/resources/settings/passwords_and_forms_page/passwords_section.html
(right):

https://codereview.chromium.org/2167043002/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/passwords_and_forms_page/passwords_section.html:54:
text-decoration: underline;
On 2016/07/21 01:59:59, Dan Beam wrote:
> i'm going to change this separately

Acknowledged.

[Dan Beam, 2016-07-21 02:16:20]

The CQ bit was checked by dbeam@chromium.org

[commit-bot: I haz the power, 2016-07-21 02:16:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-21 03:18:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-21 03:22:58]

Description was changed from

==========
MD Settings: ensure <a is="action-link"> have no :hover underline

tl;dr - the style loading order matters, action_link.css must come
before <style include="settings-shared">.

R=hcarmona@chromium.org
BUG=630067
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: ensure <a is="action-link"> have no :hover underline

tl;dr - the style loading order matters, action_link.css must come
before <style include="settings-shared">.

R=hcarmona@chromium.org
BUG=630067
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/2ff90f4a2f6f30b5e01f9bef16cbe27efce16b01
Cr-Commit-Position: refs/heads/master@{#406755}
==========

[commit-bot: I haz the power, 2016-07-21 03:22:59]

Patchset 1 (id:??) landed as
https://crrev.com/2ff90f4a2f6f30b5e01f9bef16cbe27efce16b01
Cr-Commit-Position: refs/heads/master@{#406755}