STR: 1) Click toolbar 2) Tap into toolbar to dismiss select-all 3) Long-press to select all Expected: Grey action bar with dark icons Actual: black action bar with dark grey icons Likely due to appcompat changes (bug 1220309) which landed in 48. Kevin, can you check if this affects 48 & 47?

[Tracking Requested - why for this release]: regression

Tracking for 48/49 as this is a visible issue which makes it more difficult to see the bar.

regression: 22-03 not affected 23-03 affected pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=3587b25bae302c1eed72968dbd7cef883e715948&tochange=efe7d026ac641759838dd3897c37892e37e5b244

The animation for the new Action Bar is too impressively smooth to ship in the framework so I'm going to blame AppCompat (it's also in the regression log from comment 3). We're already blocking it – bug 1220309.

Review commit: https://reviewboard.mozilla.org/r/54744/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/54744/

Comment on attachment 8755686 [details] MozReview Request: Bug 1268606 - Correct action bar style for url bar. r=liuche Review request updated; see interdiff: https://reviewboard.mozilla.org/r/54744/diff/1-2/

Comment on attachment 8755686 [details] MozReview Request: Bug 1268606 - Correct action bar style for url bar. r=liuche https://reviewboard.mozilla.org/r/54744/#review51408 Cool - do you have a summary resource for appcompat? I've just been able to find scattered links and references everywhere, for solving specific problems.

(In reply to Chenxia Liu [:liuche] from comment #7) > Cool - do you have a summary resource for appcompat? I've just been able to > find scattered links and references everywhere, for solving specific > problems. Unfortunately, no – I just tried a bunch of different combinations of attrs until it worked. StackOverflow is also *very* wrong *very* often and we do so much custom bullshit (e.g. we have a custom action bar) that many of the built-in attributes are unused. I think reading the source is your best best. See [1] for a list of all the supported "AppCompat" attributes (which often override the ones provided by the framework by removing "android:"). Then, in Intellij, you can step into the support library source with go to definition on styles to see what attrs they set & override. For this example, I overrode ThemeOverlay.AppCompat.ActionBar so I could jump into their definition & see what changed between that and ThemeOverlay.AppCompat.ActionBar.Dark (or whatever it is). Once you're in that file, you can scroll around to find other relevant AppCompat styles. [1]: http://androidxref.com/6.0.0_r1/xref/frameworks/support/v7/appcompat/res/values/attrs.xml

https://hg.mozilla.org/integration/fx-team/rev/3902c1e0359477da8daff6f805a18f7eddcb5786 Bug 1268606 - Correct action bar style for url bar. r=liuche

Comment on attachment 8755686 [details] MozReview Request: Bug 1268606 - Correct action bar style for url bar. r=liuche Approval Request Comment [Feature/regressing bug #]: bug 1220309 [User impact if declined]: Users will have an action bar inconsistently styled from the rest of our app. [Describe test coverage new/current, TreeHerder]: Tested locally on N4. [Risks and why]: Since we're just changing XML attrs, in the worst case, we style the action bar incorrectly (particularly on different device versions, but appcompat is supposed to fix that). [String/UUID change made/needed]: None

Tested using: Device: Moto X (Android 4.4.) Build: Firefox for Android 49.0a1 (2016-05-26) URL bar text selection action bar is now correctly colored: http://i.imgur.com/eUqaCAU.png But caused Bug 1275880 - "Settings" header is incorrectly colored.

Comment on attachment 8755686 [details] MozReview Request: Bug 1268606 - Correct action bar style for url bar. r=liuche Fix a regression, takign it

Release Note Request (optional, but appreciated) [Why is this notable]: [Suggested wording]: Bugfix for URL bar text selection action bar incorrectly colored [Links (documentation, blog post, etc)]:

This is a regression that was fixed... I don't think this needs a release note, since the problem was never actually in release.

Relnote - based on Comment 16.

Verified as fixed using Galaxy Note 5(6.0.1) with 48 Release and 49.0b11 Beta.