[MD Extensions] Update Detail View UI

chrome/test/data/webui/extensions/extension_detail_view_test.js

Index: chrome/test/data/webui/extensions/extension_detail_view_test.js
diff --git a/chrome/test/data/webui/extensions/extension_detail_view_test.js b/chrome/test/data/webui/extensions/extension_detail_view_test.js
index 7572b4621cf3b9d148d16b6012106a2061bff0e3..e840ff8d79e93832354e54b67c79d8d9a79fadf4 100644
--- a/chrome/test/data/webui/extensions/extension_detail_view_test.js
+++ b/chrome/test/data/webui/extensions/extension_detail_view_test.js
@@ -44,6 +44,7 @@ cr.define('extension_detail_view_tests', function() {
         item = new extensions.DetailView();
         item.set('data', extensionData);
         item.set('delegate', mockDelegate);
+        item.set('inDevMode', false);
         document.body.appendChild(item);
       });
 
@@ -54,8 +55,9 @@ cr.define('extension_detail_view_tests', function() {
 
         var testIsVisible = extension_test_util.isVisible.bind(null, item);
         expectTrue(testIsVisible('#close-button'));
-        expectFalse(testIsVisible('#open-homepage'));
-        expectFalse(testIsVisible('#options'));
+        expectTrue(testIsVisible('#icon'));
+        expectTrue(testIsVisible('#enable-toggle'));
+        expectFalse(testIsVisible('#extensions-options'));
 
         // Check the checkboxes visibility and state. They should be visible
         // only if the associated option is enabled, and checked if the
@@ -107,12 +109,22 @@ cr.define('extension_detail_view_tests', function() {
         var optionsUrl =
              'chrome-extension://' + extensionData.id + '/options.html';
         item.set('data.optionsPage', {openInTab: true, url: optionsUrl});
-        expectTrue(testIsVisible('#options'));
+        expectTrue(testIsVisible('#extensions-options'));
 
-        var homepageUrl = 'https://www.example.com/';
-        item.set('data.homePage', {specified: true, url: homepageUrl});
-        expectTrue(testIsVisible('#open-homepage'));
-        expectEquals(homepageUrl, item.$$('#open-homepage').href);
+        // TODO(devlin): Add these back once homepages are in the mocks again.

[michaelpg, 2017/03/06 19:08:46]

I would leave a TODO remove the unused code. Yay version control!

[michaelpg, 2017/03/06 19:55:02]

That is, leave this TODO, *but* remove commented-out code.

[Devlin, 2017/03/06 20:12:12]

Okay, done.

+        // expectFalse(testIsVisible('#open-homepage'));
+        // var homepageUrl = 'https://www.example.com/';
+        // item.set('data.homePage', {specified: true, url: homepageUrl});
+        // expectTrue(testIsVisible('#open-homepage'));
+        // expectEquals(homepageUrl, item.$$('#open-homepage').href);
+
+        expectFalse(testIsVisible('#id-section'));
+        expectFalse(testIsVisible('#inspectable-views'));
+
+        item.set('inDevMode', true);
+        Polymer.dom.flush();
+        expectTrue(testIsVisible('#id-section'));
+        expectTrue(testIsVisible('#inspectable-views'));
       });
 
       test(assert(TestNames.ClickableElements), function() {
@@ -133,7 +145,11 @@ cr.define('extension_detail_view_tests', function() {
             item.$$('#collect-errors'), 'setItemCollectsErrors',
             [extensionData.id, true]);
         mockDelegate.testClickingCalls(
-            item.$$('#options'), 'showItemOptionsPage', [extensionData.id]);
+            item.$$('#extensions-options'), 'showItemOptionsPage',
+            [extensionData.id]);
+        mockDelegate.testClickingCalls(
+            item.$$('#remove-extension'), 'deleteItem',
+            [extensionData.id]);
       });
     });
   }