From 80a9d2e7c9f1ebc3bed6ae6356ff1675a7c9ef62 Mon Sep 17 00:00:00 2001
From: Ray Strode <rstrode@redhat.com>
Date: Sun, 28 Aug 2011 09:29:51 -0400
Subject: [PATCH] overview: Make viewSelector private

It's only used internally by the overview itself,
and by some performance testing code, so don't
expose it as a public object.

https://bugzilla.gnome.org/show_bug.cgi?id=657082
---
 js/perf/core.js   |  4 ++--
 js/ui/overview.js | 24 ++++++++++++------------
 2 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/js/perf/core.js b/js/perf/core.js
index 7d141a2d1..4fb4fabc1 100644
--- a/js/perf/core.js
+++ b/js/perf/core.js
@@ -113,10 +113,10 @@ function run() {
 
     for (let i = 0; i < 2; i++) {
         Scripting.scriptEvent('applicationsShowStart');
-        Main.overview.viewSelector.switchTab('applications');
+        Main.overview._viewSelector.switchTab('applications');
         yield Scripting.waitLeisure();
         Scripting.scriptEvent('applicationsShowDone');
-        Main.overview.viewSelector.switchTab('windows');
+        Main.overview._viewSelector.switchTab('windows');
         yield Scripting.waitLeisure();
     }
 }
diff --git a/js/ui/overview.js b/js/ui/overview.js
index 8f33efa86..4f3d1025a 100644
--- a/js/ui/overview.js
+++ b/js/ui/overview.js
@@ -177,26 +177,26 @@ Overview.prototype = {
     init: function() {
         this.shellInfo = new ShellInfo();
 
-        this.viewSelector = new ViewSelector.ViewSelector();
-        this._group.add_actor(this.viewSelector.actor);
+        this._viewSelector = new ViewSelector.ViewSelector();
+        this._group.add_actor(this._viewSelector.actor);
 
         this._workspacesDisplay = new WorkspacesView.WorkspacesDisplay();
-        this.viewSelector.addViewTab('windows', _("Windows"), this._workspacesDisplay.actor, 'text-x-generic');
+        this._viewSelector.addViewTab('windows', _("Windows"), this._workspacesDisplay.actor, 'text-x-generic');
 
         let appView = new AppDisplay.AllAppDisplay();
-        this.viewSelector.addViewTab('applications', _("Applications"), appView.actor, 'system-run');
+        this._viewSelector.addViewTab('applications', _("Applications"), appView.actor, 'system-run');
 
         // Default search providers
-        this.viewSelector.addSearchProvider(new AppDisplay.AppSearchProvider());
-        this.viewSelector.addSearchProvider(new AppDisplay.SettingsSearchProvider());
-        this.viewSelector.addSearchProvider(new PlaceDisplay.PlaceSearchProvider());
-        this.viewSelector.addSearchProvider(new DocDisplay.DocSearchProvider());
+        this._viewSelector.addSearchProvider(new AppDisplay.AppSearchProvider());
+        this._viewSelector.addSearchProvider(new AppDisplay.SettingsSearchProvider());
+        this._viewSelector.addSearchProvider(new PlaceDisplay.PlaceSearchProvider());
+        this._viewSelector.addSearchProvider(new DocDisplay.DocSearchProvider());
 
         // TODO - recalculate everything when desktop size changes
         this.dash = new Dash.Dash();
         this._group.add_actor(this.dash.actor);
-        this.dash.actor.add_constraint(this.viewSelector.constrainY);
-        this.dash.actor.add_constraint(this.viewSelector.constrainHeight);
+        this.dash.actor.add_constraint(this._viewSelector.constrainY);
+        this.dash.actor.add_constraint(this._viewSelector.constrainHeight);
 
         // Translators: this is the name of the dock/favorites area on
         // the left of the overview
@@ -474,8 +474,8 @@ Overview.prototype = {
         }
         this.dash.actor.set_x(dashX);
 
-        this.viewSelector.actor.set_position(viewX, viewY);
-        this.viewSelector.actor.set_size(viewWidth, viewHeight);
+        this._viewSelector.actor.set_position(viewX, viewY);
+        this._viewSelector.actor.set_size(viewWidth, viewHeight);
     },
 
     //// Public methods ////