New issue
Advanced search Search tips

Issue 873823 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 15
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

[Cleanup] Remove "bool opaque" plumbed into GetTabBackgroundColor()

Project Member Reported by thomasanderson@chromium.org, Aug 13

Issue description

Project Member

Comment 1 by bugdroid1@chromium.org, Sep 15

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ec2ca8d3467833ef4a6843e74e095f7f3d11e233

commit ec2ca8d3467833ef4a6843e74e095f7f3d11e233
Author: Tom Anderson <thomasanderson@chromium.org>
Date: Sat Sep 15 02:29:30 2018

[Cleanup] [Refresh] Remove "bool opaque" plumbing for GetTabBackgroundColor()

BUG= 873823 
R=pkasting

Change-Id: Ic621aa25c9ace519b15921b4948748cefb8bee89
Reviewed-on: https://chromium-review.googlesource.com/1175413
Commit-Queue: Thomas Anderson <thomasanderson@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#591565}
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/themes/theme_properties.h
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/themes/theme_service.cc
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/themes/theme_service_aurax11.cc
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/ui/libgtkui/gtk_ui.cc
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/ui/libgtkui/gtk_ui.h
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/ui/views/frame/browser_non_client_frame_view.cc
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/ui/views/frame/browser_non_client_frame_view.h
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/ui/views/tabs/browser_tab_strip_controller.cc
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/ui/views/tabs/browser_tab_strip_controller.h
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/ui/views/tabs/fake_base_tab_strip_controller.cc
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/ui/views/tabs/fake_base_tab_strip_controller.h
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/ui/views/tabs/new_tab_button.cc
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/ui/views/tabs/new_tab_button.h
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/ui/views/tabs/tab.cc
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/ui/views/tabs/tab_controller.h
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/ui/views/tabs/tab_strip.cc
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/ui/views/tabs/tab_strip.h
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/ui/views/tabs/tab_strip_controller.h
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/chrome/browser/ui/views/tabs/tab_unittest.cc
[modify] https://crrev.com/ec2ca8d3467833ef4a6843e74e095f7f3d11e233/ui/views/linux_ui/linux_ui.h

Status: Fixed (was: Assigned)

Sign in to add a comment