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

Unified Diff: components/constrained_window/constrained_window_views_unittest.cc

Issue 2889623002: Apply View::set_preferred_size in some more places. (Closed)
Patch Set: fix SystemMenuButton Created 3 years, 7 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: components/constrained_window/constrained_window_views_unittest.cc
diff --git a/components/constrained_window/constrained_window_views_unittest.cc b/components/constrained_window/constrained_window_views_unittest.cc
index b0c98d3f4d18e52ddf03c5ff264cd69fd0406086..85a2d73edf10819965093628acbe74bd698a29b8 100644
--- a/components/constrained_window/constrained_window_views_unittest.cc
+++ b/components/constrained_window/constrained_window_views_unittest.cc
@@ -32,22 +32,16 @@ class DialogContents : public views::DialogDelegateView {
DialogContents() {}
~DialogContents() override {}
- void set_preferred_size(const gfx::Size& preferred_size) {
- preferred_size_ = preferred_size;
- }
-
void set_modal_type(ui::ModalType modal_type) { modal_type_ = modal_type; }
// DialogDelegateView:
views::View* GetContentsView() override { return this; }
- gfx::Size GetPreferredSize() const override { return preferred_size_; }
gfx::Size GetMinimumSize() const override { return gfx::Size(); }
// WidgetDelegate:
ui::ModalType GetModalType() const override { return modal_type_; }
private:
- gfx::Size preferred_size_;
ui::ModalType modal_type_ = ui::MODAL_TYPE_NONE;
DISALLOW_COPY_AND_ASSIGN(DialogContents);

Powered by Google App Engine
This is Rietveld 408576698