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

Unified Diff: ui/app_list/views/app_list_view.cc

Issue 309443002: Remove need to sign in to use the App Launcher. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Compile Created 6 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
« no previous file with comments | « ui/app_list/test/app_list_test_view_delegate.cc ('k') | ui/app_list/views/signin_view.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/app_list/views/app_list_view.cc
diff --git a/ui/app_list/views/app_list_view.cc b/ui/app_list/views/app_list_view.cc
index e1da9a150cde08af5f355a71d37fcc411e989f74..c3525119d24deac95235aad385bc13462148d56a 100644
--- a/ui/app_list/views/app_list_view.cc
+++ b/ui/app_list/views/app_list_view.cc
@@ -13,7 +13,6 @@
#include "ui/app_list/app_list_model.h"
#include "ui/app_list/app_list_view_delegate.h"
#include "ui/app_list/pagination_model.h"
-#include "ui/app_list/signin_delegate.h"
#include "ui/app_list/speech_ui_model.h"
#include "ui/app_list/views/app_list_background.h"
#include "ui/app_list/views/app_list_folder_view.h"
@@ -22,7 +21,6 @@
#include "ui/app_list/views/apps_container_view.h"
#include "ui/app_list/views/contents_view.h"
#include "ui/app_list/views/search_box_view.h"
-#include "ui/app_list/views/signin_view.h"
#include "ui/app_list/views/speech_view.h"
#include "ui/base/ui_base_switches.h"
#include "ui/compositor/layer.h"
@@ -148,7 +146,6 @@ class HideViewAnimationObserver : public ui::ImplicitAnimationObserver {
AppListView::AppListView(AppListViewDelegate* delegate)
: delegate_(delegate),
app_list_main_view_(NULL),
- signin_view_(NULL),
calamity 2014/06/02 04:36:15 This file needs corresponding changes to the heade
benwells 2014/06/02 07:48:55 Done.
speech_view_(NULL),
overlay_view_(NULL),
animation_observer_(new HideViewAnimationObserver()) {
@@ -254,12 +251,6 @@ void AppListView::Prerender() {
}
void AppListView::OnProfilesChanged() {
- SigninDelegate* signin_delegate =
- delegate_ ? delegate_->GetSigninDelegate() : NULL;
- bool show_signin_view = signin_delegate && signin_delegate->NeedSignin();
-
- signin_view_->SetVisible(show_signin_view);
- app_list_main_view_->SetVisible(!show_signin_view);
app_list_main_view_->search_box_view()->InvalidateMenu();
}
@@ -308,11 +299,6 @@ void AppListView::InitAsBubbleInternal(gfx::NativeView parent,
app_list_main_view_->layer()->SetMasksToBounds(true);
#endif
- signin_view_ =
- new SigninView(delegate_->GetSigninDelegate(),
- app_list_main_view_->GetPreferredSize().width());
- AddChildView(signin_view_);
-
// Speech recognition is available only when the start page exists.
if (delegate_ && delegate_->GetSpeechRecognitionContents()) {
speech_view_ = new SpeechView(delegate_.get());
@@ -459,7 +445,6 @@ bool AppListView::AcceleratorPressed(const ui::Accelerator& accelerator) {
void AppListView::Layout() {
const gfx::Rect contents_bounds = GetContentsBounds();
app_list_main_view_->SetBoundsRect(contents_bounds);
- signin_view_->SetBoundsRect(contents_bounds);
if (speech_view_) {
gfx::Rect speech_bounds = contents_bounds;
@@ -510,7 +495,7 @@ void AppListView::OnWidgetVisibilityChanged(views::Widget* widget,
void AppListView::OnSpeechRecognitionStateChanged(
SpeechRecognitionState new_state) {
- if (signin_view_->visible() || !speech_view_)
+ if (!speech_view_)
return;
bool recognizing = (new_state == SPEECH_RECOGNITION_RECOGNIZING ||
« no previous file with comments | « ui/app_list/test/app_list_test_view_delegate.cc ('k') | ui/app_list/views/signin_view.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698