My favorites | Sign in
Project Home Downloads Wiki Issues Code Search
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 175426: The tab background flashes when resizing the browser window with certain themes
1 person starred this issue and may be notified of changes. Back to list
Status:  Verified
Owner:  pkotw...@chromium.org
Closed:  Feb 2013


Sign in to add a comment
 
Project Member Reported by pkotw...@chromium.org, Feb 11, 2013
The tab background flashes when resizing the browser window with certain themes

Repro steps:
1) Install https://chrome.google.com/webstore/detail/lamborghini/fiefegoncbfdemobfpaldfapbfiinmeo?hl=EN
2) Open 10 tabs
3) Resize the window

Expected:
- The inactive tabs are semi transparent. The background frame is visible from underneath the inactive tabs. Nothing special occurs while resizing the window
Actual:
- Sit back and enjoy the light show
Feb 12, 2013
#1 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=182095

------------------------------------------------------------------------
r182095 | pkotwicz@chromium.org | 2013-02-13T02:33:17.503415Z

Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/ui/views/tabs/tab.cc?r1=182095&r2=182094&pathrev=182095

Do not cache the tab background if a tab background was autogenerated by a theme

BUG=175426
Test=None

Review URL: https://chromiumcodereview.appspot.com/12218093
------------------------------------------------------------------------
Feb 13, 2013
#2 pkotw...@chromium.org
(No comment was entered for this change.)
Labels: Merge-Requested
Feb 15, 2013
#3 gir...@chromium.org
Low risk patch, artifacts are visible to all users with themes (about 20% of the user base.)
Status: Started
Feb 15, 2013
#4 dhar...@chromium.org
once this is verified in canary, I'll approve the merge.
Feb 18, 2013
#5 dhar...@chromium.org
(No comment was entered for this change.)
Labels: -Merge-Requested Merge-Approved
Feb 19, 2013
#6 gir...@chromium.org
(No comment was entered for this change.)
Labels: Iteration-76
Feb 19, 2013
#7 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=183223

------------------------------------------------------------------------
r183223 | pkotwicz@chromium.org | 2013-02-19T15:58:34.649870Z

Changed paths:
   M http://src.chromium.org/viewvc/chrome/branches/1410/src/chrome/browser/ui/views/tabs/tab.cc?r1=183223&r2=183222&pathrev=183223

Merge 182095
> Do not cache the tab background if a tab background was autogenerated by a theme
> 
> BUG=175426
> Test=None
> 
> Review URL: https://chromiumcodereview.appspot.com/12218093

TBR=pkotwicz@chromium.org
Review URL: https://codereview.chromium.org/12302027
------------------------------------------------------------------------
Labels: -Merge-Approved merge-merged-1410
Feb 19, 2013
#8 pkotw...@chromium.org
(No comment was entered for this change.)
Status: Fixed
Mar 7, 2013
#9 binch...@chromium.org
Platform 26.0.1410.29  beta
ChromeOS 3701.43.0     beta-channel

Status: Verified
Mar 10, 2013
#10 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: -Area-UI -Feature-Themes -Mstone-26 Cr-UI-Browser-Themes M-26 Cr-UI
Sign in to add a comment

Powered by Google Project Hosting