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

Issue 10700144: Merge 145727 - Fix fade-truncation of LTR tab titles under RTL UI. (Closed)

Created:
8 years, 5 months ago by Alexei Svitkine (slow)
Modified:
8 years, 5 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Merge 145727 - Fix fade-truncation of LTR tab titles under RTL UI. This was regressed by http://crrev.com/145435, which made UpdateRenderText() get a default alignment based on the UI directionality in the case where |flags| didn't specify one. This caused an issue for DrawFadeTruncatingString(), which assumed that the default alignment would LTR unless specified otherwise. This CL fixes the issue by having DrawFadeTruncatingString() specify LTR alignment whenever it doesn't explicitly set it to RTL, which matches its behavior before r145435. BUG=135639 TEST=Open chrome.exe --lang=he and go to amazon.com. The tab title, which should be in English, should be fade truncated on the right end. Go to http://www.haaretz.co.il and text should be fade truncated on the left end. When launched in LTR mode (chrome.exe --lang=en), both of those tab titles should be fade-truncated on right, per the M19 behavior. Review URL: https://chromiumcodereview.appspot.com/10743002 TBR=asvitkine@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=145918

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M ui/gfx/canvas_skia.cc View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Alexei Svitkine (slow)
8 years, 5 months ago (2012-07-10 18:10:11 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698