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

Issue 2573383002: Refresh appearance of omnibox results when native theme changes. (Closed)

Created:
4 years ago by Evan Stade
Modified:
4 years ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2924
Project:
chromium
Visibility:
Public.

Description

Refresh appearance of omnibox results when native theme changes. Previous to this patch, the render texts were created just once. In some cases that was before the view was actually added to a widget hierarchy and therefore GetNativeTheme returned the wrong thing. This only affects GTK theme mode because on all other platforms, there's only ever one NativeTheme. (Incognito does use a different NativeTheme as well but this only affected suggestions which we don't provide in incognito.) BUG=672226 Review-Url: https://codereview.chromium.org/2578443002 Cr-Commit-Position: refs/heads/master@{#438418} (cherry picked from commit cfd026f99e1868040b970891ba75d9394c5b8efa) Committed: https://chromium.googlesource.com/chromium/src/+/9d6ad781e3ec3ec65eb7c656ef10f18d80ed9b0b

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M chrome/browser/ui/views/omnibox/omnibox_result_view.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/ui/views/omnibox/omnibox_result_view.cc View 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
Evan Stade
4 years ago (2016-12-15 03:14:48 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
9d6ad781e3ec3ec65eb7c656ef10f18d80ed9b0b.

Powered by Google App Engine
This is Rietveld 408576698