Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(292)

Issue 2849283002: Fix omnibox suggestions being overdrawn with the nav bar. (Closed)

Created:
3 years, 7 months ago by Ted C
Modified:
3 years, 7 months ago
Reviewers:
mdjones
CC:
chromium-reviews, jdonnelly+watch_chromium.org, agrieve+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Fix omnibox suggestions being overdrawn with the nav bar. getLocationInWindow adjusts for the Android nav bar, while the positioning of the parent is exclusive of the nav bar. This causes the suggestions be too bar left aligned and cut off. Only visible if you have the nav bar aligned with the start of the text (i.e. on the left with LTR text). BUG=669150 Review-Url: https://codereview.chromium.org/2849283002 Cr-Commit-Position: refs/heads/master@{#468466} Committed: https://chromium.googlesource.com/chromium/src/+/c0fd363129a9a01abe8954eb9b23beadaf6ea984

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -3 lines) Patch
M chrome/android/java/src/org/chromium/chrome/browser/omnibox/SuggestionView.java View 4 chunks +7 lines, -3 lines 0 comments Download

Messages

Total messages: 11 (5 generated)
Ted C
PTAL
3 years, 7 months ago (2017-05-01 21:25:39 UTC) #2
mdjones
"cases" -> "causes" in description, otherwise lgtm
3 years, 7 months ago (2017-05-01 21:43:23 UTC) #3
Ted C
On 2017/05/01 21:43:23, mdjones wrote: > "cases" -> "causes" in description, otherwise lgtm Done
3 years, 7 months ago (2017-05-01 21:51:38 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2849283002/1
3 years, 7 months ago (2017-05-01 21:52:39 UTC) #7
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://chromium.googlesource.com/chromium/src/+/c0fd363129a9a01abe8954eb9b23beadaf6ea984
3 years, 7 months ago (2017-05-01 22:47:41 UTC) #10
gone
3 years, 7 months ago (2017-05-02 05:29:25 UTC) #11
Message was sent while issue was closed.
A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2853993002/ by dfalcantara@chromium.org.

The reason for reverting is: Caused
https://bugs.chromium.org/p/chromium/issues/detail?id=717395

Got the go ahead to revert over email.  Will continue investigating what's going
on in the bug as a follow up..

Powered by Google App Engine
This is Rietveld 408576698