Fixed "Relaunch and Powerwash" button on a chrome://help page.

chrome/browser/resources/help/channel_change_page.js

Index: chrome/browser/resources/help/channel_change_page.js
diff --git a/chrome/browser/resources/help/channel_change_page.js b/chrome/browser/resources/help/channel_change_page.js
index b084effeec39b984d42d466c53add65be0f720eb..3763f6533c797e2dd53f080bc0836a9671603e10 100644
--- a/chrome/browser/resources/help/channel_change_page.js
+++ b/chrome/browser/resources/help/channel_change_page.js
@@ -195,7 +195,6 @@ cr.define('help', function() {
      */
     updateIsEnterpriseManaged_: function(isEnterpriseManaged) {
       this.isEnterpriseManaged_ = isEnterpriseManaged;
-      help.HelpPage.updateChannelChangePageContainerVisibility();

[michaelpg, 2014/09/15 19:05:52]

Is this call replaced anywhere?

this works because we call updateIsEnterpriseManaged before we call
update[Current|Target]Channel, but it's not clear that this page should be
dependent on that ordering.

[ygorshenin1, 2014/09/16 12:49:47]

Thanks for pointing this out! I've added HelpPage.updateIsEnterpriseManaged()
which implicitly calls updateChannelChangePageContainerVisibility() (as it was
done for setTargetChannel() and setCurrentChannel()).

On 2014/09/15 19:05:52, michaelpg wrote:

     },
 
     /**
@@ -209,7 +208,6 @@ cr.define('help', function() {
         return;
       this.currentChannel_ = channel;
       this.selectOption_(channel);
-      help.HelpPage.updateChannelChangePageContainerVisibility();
     },
 
     /**
@@ -223,7 +221,6 @@ cr.define('help', function() {
       if (this.channelList_.indexOf(channel) < 0)
         return;
       this.targetChannel_ = channel;
-      help.HelpPage.updateChannelChangePageContainerVisibility();
     },
 
     /**