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

Side by Side Diff: ui/app_list/views/contents_view.cc

Issue 2865553002: Naming more views and windows to simplify debugging. (Closed)
Patch Set: 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "ui/app_list/views/contents_view.h" 5 #include "ui/app_list/views/contents_view.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
42 views::BoxLayout::MAIN_AXIS_ALIGNMENT_CENTER); 42 views::BoxLayout::MAIN_AXIS_ALIGNMENT_CENTER);
43 SetLayoutManager(answer_container_layout); 43 SetLayoutManager(answer_container_layout);
44 } 44 }
45 45
46 // views::View overrides: 46 // views::View overrides:
47 void ChildPreferredSizeChanged(View* child) override { 47 void ChildPreferredSizeChanged(View* child) override {
48 if (visible()) 48 if (visible())
49 search_results_page_view_->Layout(); 49 search_results_page_view_->Layout();
50 } 50 }
51 51
52 const char* GetClassName() const override {
53 return "SearchAnswerContainerView";
54 }
55
52 private: 56 private:
53 views::View* const search_results_page_view_; 57 views::View* const search_results_page_view_;
54 58
55 DISALLOW_COPY_AND_ASSIGN(SearchAnswerContainerView); 59 DISALLOW_COPY_AND_ASSIGN(SearchAnswerContainerView);
56 }; 60 };
57 61
58 } // namespace 62 } // namespace
59 63
60 ContentsView::ContentsView(AppListMainView* app_list_main_view) 64 ContentsView::ContentsView(AppListMainView* app_list_main_view)
61 : model_(nullptr), 65 : model_(nullptr),
(...skipping 461 matching lines...) Expand 10 before | Expand all | Expand 10 after
523 527
524 void ContentsView::OnSearchAnswerAvailableChanged(bool has_answer) { 528 void ContentsView::OnSearchAnswerAvailableChanged(bool has_answer) {
525 if (has_answer == search_answer_container_view_->visible()) 529 if (has_answer == search_answer_container_view_->visible())
526 return; 530 return;
527 531
528 search_answer_container_view_->SetVisible(has_answer); 532 search_answer_container_view_->SetVisible(has_answer);
529 search_results_page_view_->Layout(); 533 search_results_page_view_->Layout();
530 } 534 }
531 535
532 } // namespace app_list 536 } // namespace app_list
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698