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

Side by Side Diff: chrome/browser/ui/views/options/passwords_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/passwords_page_view.h" 5 #include "chrome/browser/ui/views/options/passwords_page_view.h"
6 6
7 #include "app/l10n_util.h" 7 #include "app/l10n_util.h"
8 #include "base/i18n/rtl.h" 8 #include "base/i18n/rtl.h"
9 #include "base/string_util.h" 9 #include "base/string_util.h"
10 #include "base/utf_string_conversions.h" 10 #include "base/utf_string_conversions.h"
(...skipping 259 matching lines...) Expand 10 before | Expand all | Expand 10 after
270 } 270 }
271 271
272 /////////////////////////////////////////////////////////////////////////////// 272 ///////////////////////////////////////////////////////////////////////////////
273 // PasswordsPageView, protected 273 // PasswordsPageView, protected
274 void PasswordsPageView::InitControlLayout() { 274 void PasswordsPageView::InitControlLayout() {
275 SetupButtonsAndLabels(); 275 SetupButtonsAndLabels();
276 SetupTable(); 276 SetupTable();
277 277
278 // Do the layout thing. 278 // Do the layout thing.
279 const int top_column_set_id = 0; 279 const int top_column_set_id = 0;
280 GridLayout* layout = CreatePanelGridLayout(this); 280 GridLayout* layout = GridLayout::CreatePanel(this);
281 SetLayoutManager(layout); 281 SetLayoutManager(layout);
282 282
283 // Design the grid. 283 // Design the grid.
284 ColumnSet* column_set = layout->AddColumnSet(top_column_set_id); 284 ColumnSet* column_set = layout->AddColumnSet(top_column_set_id);
285 column_set->AddColumn(GridLayout::FILL, GridLayout::FILL, 1, 285 column_set->AddColumn(GridLayout::FILL, GridLayout::FILL, 1,
286 GridLayout::USE_PREF, 0, 0); 286 GridLayout::USE_PREF, 0, 0);
287 column_set->AddPaddingColumn(0, kRelatedControlHorizontalSpacing); 287 column_set->AddPaddingColumn(0, kRelatedControlHorizontalSpacing);
288 column_set->AddColumn(GridLayout::FILL, GridLayout::CENTER, 0, 288 column_set->AddColumn(GridLayout::FILL, GridLayout::CENTER, 0,
289 GridLayout::USE_PREF, 0, 0); 289 GridLayout::USE_PREF, 0, 0);
290 290
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
362 bool show = allow_show_passwords_.GetValue(); 362 bool show = allow_show_passwords_.GetValue();
363 if (!show) 363 if (!show)
364 HidePassword(); 364 HidePassword();
365 show_button_.SetVisible(show); 365 show_button_.SetVisible(show);
366 password_label_.SetVisible(show); 366 password_label_.SetVisible(show);
367 // Update the layout (it may depend on the button size). 367 // Update the layout (it may depend on the button size).
368 show_button_.InvalidateLayout(); 368 show_button_.InvalidateLayout();
369 Layout(); 369 Layout();
370 } 370 }
371 } 371 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/options/languages_page_view.cc ('k') | chrome/browser/ui/views/uninstall_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698