Review feedback.

app/subapps/browser/browser.js

@@ -238,8 +238,6 @@ YUI.add('subapp-browser', function(Y) {
     */
     _charmBrowserDispatcher: function(metadata) {
       this.renderCharmBrowser(metadata);
-      // XXX Won't be needed once window.flags.il becomes the norm. The details
-      // template should be updated to hide by default.
       if (this._shouldShowCharm()) {
         // The entity rendering views need to handle the new state format
         // before this can be hooked up.

app/subapps/browser/views/addedservices.js

@@ -32,9 +32,6 @@ YUI.add('juju-addedservices', function(Y) {
     views.SearchWidgetMgmtExtension
   ], {
 
-    events: {
-    },
-
     template: Templates.addedservices,
 
     /**
@@ -54,8 +51,6 @@ YUI.add('juju-addedservices', function(Y) {
       @method render
     */
     render: function() {
-      // To keep render idempotent we check to see if any changes have been
-      // made since the last time it's been rendered.
       var container = this.get('container');
       container.setHTML(this.template());
       // Provided by 'search-widget-mgmt-extension'.

test/test_addedservices_view.js

@@ -43,6 +43,10 @@ describe('added services view', function() {
     }
   });
 
+  it('is extended by the search widget', function() {
+    assert.notEqual(view._renderSearchWidget, undefined);
+  });
+
   describe('render', function() {
     var renderSearch;
 
@@ -66,9 +70,6 @@ describe('added services view', function() {
 
   describe('destroy', function() {
 
-    beforeEach(function() {
-    });
-
     it('removes the container from the DOM', function() {
       view.render();
       var container = view.get('container');

test/test_browser_app.js

@@ -582,17 +582,29 @@ with this program.  If not, see <http://www.gnu.org/licenses/>.
         });
 
         describe('_addedServicesDispatcher', function() {
+          var detailsVisible;
+
           function stubRenderers(context) {
             renderAddedServices = utils.makeStubMethod(
                 app, 'renderAddedServices');
             context._cleanups.push(renderAddedServices.reset);
+            detailsVisible = utils.makeStubMethod(
+                app, '_detailsVisible');
+            context._cleanups.push(detailsVisible.reset);
           }
 
           it('calls to render the added services view', function() {
             stubRenderers(this);
             app._addedServicesDispatcher();
             assert.equal(renderAddedServices.callCount(), 1);
           });
+
+          it('hides the details panel', function() {
+            stubRenderers(this);
+            app._addedServicesDispatcher();
+            assert.equal(detailsVisible.callCount(), 1);
+            assert.equal(detailsVisible.lastArguments()[0], false);
+          });
         });