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

Unified Diff: chrome/browser/ui/views/collected_cookies_views.cc

Issue 63343004: Remove DialogDelegate::UseNewStyle and unused code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Restore [D]CHECKs and loop scoping. Created 7 years, 1 month 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/views/collected_cookies_views.cc
diff --git a/chrome/browser/ui/views/collected_cookies_views.cc b/chrome/browser/ui/views/collected_cookies_views.cc
index 9fceea8e0933810db05c7478f3c5ca829970e307..30d422e33e00a23ea0763befdadcc03013928313 100644
--- a/chrome/browser/ui/views/collected_cookies_views.cc
+++ b/chrome/browser/ui/views/collected_cookies_views.cc
@@ -29,7 +29,6 @@
#include "content/public/browser/notification_details.h"
#include "content/public/browser/notification_source.h"
#include "content/public/browser/web_contents.h"
-#include "content/public/browser/web_contents_view.h"
#include "grit/generated_resources.h"
#include "grit/locale_settings.h"
#include "grit/theme_resources.h"
@@ -215,9 +214,7 @@ CollectedCookiesViews::CollectedCookiesViews(content::WebContents* web_contents)
web_contents_modal_dialog_manager->delegate();
DCHECK(modal_delegate);
window_ = views::Widget::CreateWindowAsFramelessChild(
- this,
- web_contents->GetView()->GetNativeView(),
- modal_delegate->GetWebContentsModalDialogHost()->GetHostView());
+ this, modal_delegate->GetWebContentsModalDialogHost()->GetHostView());
web_contents_modal_dialog_manager->ShowDialog(window_->GetNativeView());
}
@@ -331,13 +328,8 @@ void CollectedCookiesViews::Init() {
GridLayout::USE_PREF, 0, 0);
layout->StartRow(0, single_column_layout_id);
- views::TabbedPane* tabbed_pane = NULL;
- if (DialogDelegate::UseNewStyle()) {
- tabbed_pane = new views::TabbedPane(false);
- layout->SetInsets(gfx::Insets(kTabbedPaneTopPadding, 0, 0, 0));
- } else {
- tabbed_pane = new views::TabbedPane(true);
- }
+ views::TabbedPane* tabbed_pane = new views::TabbedPane();
+ layout->SetInsets(gfx::Insets(kTabbedPaneTopPadding, 0, 0, 0));
layout->AddView(tabbed_pane);
// NOTE: Panes must be added after |tabbed_pane| has been added to its parent.
@@ -354,9 +346,7 @@ void CollectedCookiesViews::Init() {
layout->StartRow(0, single_column_layout_id);
cookie_info_view_ = new CookieInfoView();
layout->AddView(cookie_info_view_);
- layout->AddPaddingRow(0, DialogDelegate::UseNewStyle() ?
- kCookieInfoBottomPadding :
- views::kRelatedControlVerticalSpacing);
+ layout->AddPaddingRow(0, kCookieInfoBottomPadding);
layout->StartRow(0, single_column_layout_id);
infobar_ = new InfobarView();
@@ -394,10 +384,8 @@ views::View* CollectedCookiesViews::CreateAllowedPane() {
views::View* pane = new views::View();
GridLayout* layout = GridLayout::CreatePanel(pane);
- if (DialogDelegate::UseNewStyle()) {
- layout->SetInsets(kVPanelPadding, views::kButtonHEdgeMarginNew,
- kVPanelPadding, views::kButtonHEdgeMarginNew);
- }
+ layout->SetInsets(kVPanelPadding, views::kButtonHEdgeMarginNew,
+ kVPanelPadding, views::kButtonHEdgeMarginNew);
pane->SetLayoutManager(layout);
const int single_column_layout_id = 0;
@@ -407,9 +395,7 @@ views::View* CollectedCookiesViews::CreateAllowedPane() {
layout->StartRow(0, single_column_layout_id);
layout->AddView(allowed_label_);
- layout->AddPaddingRow(0, DialogDelegate::UseNewStyle() ?
- kLabelBottomPadding :
- views::kRelatedControlVerticalSpacing);
+ layout->AddPaddingRow(0, kLabelBottomPadding);
layout->StartRow(1, single_column_layout_id);
layout->AddView(CreateScrollView(allowed_cookies_tree_), 1, 1,
@@ -463,10 +449,8 @@ views::View* CollectedCookiesViews::CreateBlockedPane() {
views::View* pane = new views::View();
GridLayout* layout = GridLayout::CreatePanel(pane);
- if (DialogDelegate::UseNewStyle()) {
- layout->SetInsets(kVPanelPadding, views::kButtonHEdgeMarginNew,
- kVPanelPadding, views::kButtonHEdgeMarginNew);
- }
+ layout->SetInsets(kVPanelPadding, views::kButtonHEdgeMarginNew,
+ kVPanelPadding, views::kButtonHEdgeMarginNew);
pane->SetLayoutManager(layout);
const int single_column_layout_id = 0;
@@ -484,9 +468,7 @@ views::View* CollectedCookiesViews::CreateBlockedPane() {
layout->StartRow(0, single_column_layout_id);
layout->AddView(blocked_label_, 1, 1, GridLayout::FILL, GridLayout::FILL);
- layout->AddPaddingRow(0, DialogDelegate::UseNewStyle() ?
- kLabelBottomPadding :
- views::kRelatedControlVerticalSpacing);
+ layout->AddPaddingRow(0, kLabelBottomPadding);
layout->StartRow(1, single_column_layout_id);
layout->AddView(
@@ -502,15 +484,11 @@ views::View* CollectedCookiesViews::CreateBlockedPane() {
}
views::View* CollectedCookiesViews::CreateScrollView(views::TreeView* pane) {
- if (DialogDelegate::UseNewStyle()) {
- views::ScrollView* scroll_view = new views::ScrollView();
- scroll_view->SetContents(pane);
- scroll_view->set_border(
- views::Border::CreateSolidBorder(1, kCookiesBorderColor));
- return scroll_view;
- } else {
- return pane->CreateParentIfNecessary();
- }
+ views::ScrollView* scroll_view = new views::ScrollView();
+ scroll_view->SetContents(pane);
+ scroll_view->set_border(
+ views::Border::CreateSolidBorder(1, kCookiesBorderColor));
+ return scroll_view;
}
void CollectedCookiesViews::EnableControls() {
« no previous file with comments | « chrome/browser/ui/views/bookmarks/bookmark_editor_view.cc ('k') | chrome/browser/ui/views/color_chooser_aura.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698