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

Side by Side Diff: chrome/browser/ui/views/options/exceptions_page_view.cc

Issue 6384002: views: Make CreatePanelGridLayout a static method of GridLayout class. (Closed)
Patch Set: fix trybots Created 9 years, 11 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/views/options/exceptions_page_view.h" 5 #include "chrome/browser/ui/views/options/exceptions_page_view.h"
6 6
7 #include "app/l10n_util.h" 7 #include "app/l10n_util.h"
8 #include "base/string_util.h" 8 #include "base/string_util.h"
9 #include "base/utf_string_conversions.h" 9 #include "base/utf_string_conversions.h"
10 #include "chrome/browser/prefs/pref_service.h" 10 #include "chrome/browser/prefs/pref_service.h"
(...skipping 111 matching lines...) Expand 10 before | Expand all | Expand 10 after
122 remove_all_button_.SetEnabled(rows > 0); 122 remove_all_button_.SetEnabled(rows > 0);
123 } 123 }
124 124
125 /////////////////////////////////////////////////////////////////////////////// 125 ///////////////////////////////////////////////////////////////////////////////
126 // ExceptionsPageView, protected 126 // ExceptionsPageView, protected
127 void ExceptionsPageView::InitControlLayout() { 127 void ExceptionsPageView::InitControlLayout() {
128 SetupButtons(); 128 SetupButtons();
129 SetupTable(); 129 SetupTable();
130 130
131 // Do the layout thing. 131 // Do the layout thing.
132 GridLayout* layout = CreatePanelGridLayout(this); 132 GridLayout* layout = GridLayout::CreatePanel(this);
133 SetLayoutManager(layout); 133 SetLayoutManager(layout);
134 134
135 const int top_column_set_id = 0; 135 const int top_column_set_id = 0;
136 136
137 // Design the grid. 137 // Design the grid.
138 ColumnSet* column_set = layout->AddColumnSet(top_column_set_id); 138 ColumnSet* column_set = layout->AddColumnSet(top_column_set_id);
139 column_set->AddColumn(GridLayout::FILL, GridLayout::FILL, 1, 139 column_set->AddColumn(GridLayout::FILL, GridLayout::FILL, 1,
140 GridLayout::USE_PREF, 0, 0); 140 GridLayout::USE_PREF, 0, 0);
141 column_set->AddPaddingColumn(0, kRelatedControlHorizontalSpacing); 141 column_set->AddPaddingColumn(0, kRelatedControlHorizontalSpacing);
142 column_set->AddColumn(GridLayout::FILL, GridLayout::CENTER, 0, 142 column_set->AddColumn(GridLayout::FILL, GridLayout::CENTER, 0,
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
189 columns.back().sortable = true; 189 columns.back().sortable = true;
190 table_view_ = new views::TableView(&table_model_, columns, views::TEXT_ONLY, 190 table_view_ = new views::TableView(&table_model_, columns, views::TEXT_ONLY,
191 true, true, true); 191 true, true, true);
192 // Make the table initially sorted by host. 192 // Make the table initially sorted by host.
193 views::TableView::SortDescriptors sort; 193 views::TableView::SortDescriptors sort;
194 sort.push_back(views::TableView::SortDescriptor( 194 sort.push_back(views::TableView::SortDescriptor(
195 IDS_PASSWORDS_PAGE_VIEW_SITE_COLUMN, true)); 195 IDS_PASSWORDS_PAGE_VIEW_SITE_COLUMN, true));
196 table_view_->SetSortDescriptors(sort); 196 table_view_->SetSortDescriptors(sort);
197 table_view_->SetObserver(this); 197 table_view_->SetObserver(this);
198 } 198 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/options/exception_editor_view.cc ('k') | chrome/browser/ui/views/options/fonts_page_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698