DescriptionAdd a separate param for https scheme hightlight to emphasizeUrl
Currently the utility call for emphasizing urls uses useDarkText
as a signal to decide whether https scheme should be colored. Since
with theme colors this logic gets more compicated, this change adds
another explicit param to control the coloring of https.
BUG=482780
Committed: https://crrev.com/dc364e9a13403cbc7b7eed3239baa463256b9ca4
Cr-Commit-Position: refs/heads/master@{#330183}
Patch Set 1 #
Total comments: 1
Patch Set 2 : Renamed param and removed security state check to caller #Patch Set 3 : Assert that emphasizeHttps doesn't violate security state concerns #Patch Set 4 : Fix tests #
Messages
Total messages: 15 (6 generated)
|