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

Unified Diff: chrome/browser/ui/panels/panel_browser_view.cc

Issue 8221027: Make views::Label and views::Link auto-color themselves to be readable over their background colo... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/panels/panel_browser_view.cc
===================================================================
--- chrome/browser/ui/panels/panel_browser_view.cc (revision 104959)
+++ chrome/browser/ui/panels/panel_browser_view.cc (working copy)
@@ -495,6 +495,8 @@
NativePanelTestingWin::NativePanelTestingWin(
PanelBrowserView* panel_browser_view) :
panel_browser_view_(panel_browser_view) {
+ PanelBrowserFrameView* frame_view = panel_browser_view_->GetFrameView();
+ frame_view->title_label_->SetAutoColorReadabilityEnabled(false);
}
void NativePanelTestingWin::PressLeftMouseButtonTitlebar(
@@ -527,7 +529,7 @@
PanelBrowserFrameView* frame_view = panel_browser_view_->GetFrameView();
SkColor attention_color = frame_view->GetTitleColor(
PanelBrowserFrameView::PAINT_FOR_ATTENTION);
- return attention_color == frame_view->title_label_->GetColor();
+ return attention_color == frame_view->title_label_->enabled_color();
}
bool NativePanelTestingWin::VerifyActiveState(bool is_active) {
@@ -540,7 +542,7 @@
return false;
SkColor expected_color = frame_view->GetTitleColor(expected_paint_state);
- return expected_color == frame_view->title_label_->GetColor();
+ return expected_color == frame_view->title_label_->enabled_color();
}
bool NativePanelTestingWin::IsWindowSizeKnown() const {
« no previous file with comments | « chrome/browser/ui/panels/panel_browser_frame_view.cc ('k') | chrome/browser/ui/panels/panel_browser_view_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698