Hide the DIV containing password import/export buttons

chrome/browser/resources/options/password_manager.js

Index: chrome/browser/resources/options/password_manager.js
diff --git a/chrome/browser/resources/options/password_manager.js b/chrome/browser/resources/options/password_manager.js
index 0cf7d229b8fc23ff56046ded0c93bb4c12c16cef..6c79c51936c2a7550fba04bdfc0584a953e03f2d 100644
--- a/chrome/browser/resources/options/password_manager.js
+++ b/chrome/browser/resources/options/password_manager.js
@@ -289,8 +289,7 @@ cr.define('options', function() {
 
     /** @private */
     showImportExportButton_: function() {
-      $('password-manager-import').hidden = false;
-      $('password-manager-export').hidden = false;
+      $('password-manager-import-export').style.display = '-webkit-box';

[vabr (Chromium), 2016/07/18 14:07:17]

-webkit-box is what the action-area class in ui/webui/resources/css/overlay.css
uses. The DIV has this class, so I'm emulating dropping the override from the
password_manager.css class here. It seems fragile because if someone updates
action-area, this line will likely be missed. But I'm not sure how to do this
better.

[stevenjb, 2016/07/18 15:27:18]

Instead can you toggle a class, e.g. 'noDisplay' and change the CSS to something
like #password-manager-import-export.noDisplay { display: none } ?

[vabr (Chromium), 2016/07/18 15:53:10]

Good idea, done.

If you prefer "noDisplay" to the "hidden" which I used, just let me know and
I'll rename it. In that case, please confirm that it should be "noDisplay" and
not "no-display" (the latter style seems to be used for the class names in the
code around).

[stevenjb, 2016/07/18 16:34:33]

Since there is already a global 'hidden' attribute, with subtly different
behavior (as demonstrated in the bug this is fixing), using a class with the
same name seems confusing. You are right though, classes should be named, e.g.
no-display.

[vabr (Chromium), 2016/07/19 08:16:57]

Acknowledged.
(Would implement that, but your other suggestion with the hidden attribute
worked even better.)

     },
   };