Force a rescan when opening the Wifi menu on the settings page.

Force a rescan when opening the Wifi menu on the settings page.

BUG=123030
TEST=Manual.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=133820

[kevers, 2012-04-23 20:16:31]

Hi Chris,

Can you please have a look at this CL.

[csilv, 2012-04-24 17:15:54]

lgtm

http://codereview.chromium.org/10175010/diff/1/chrome/browser/ui/webui/option...
File chrome/browser/ui/webui/options2/chromeos/internet_options_handler2.cc
(right):

http://codereview.chromium.org/10175010/diff/1/chrome/browser/ui/webui/option...
chrome/browser/ui/webui/options2/chromeos/internet_options_handler2.cc:439:
web_ui()->RegisterMessageCallback("buttonClickCallback",
optional: I think the message strings would be better as "refreshNetworks" and
"buttonClick" (ie, omit "Callback" in the message name.) Consistent with the
rest of this method.

[kevers, 2012-04-24 17:46:13]

http://codereview.chromium.org/10175010/diff/1/chrome/browser/ui/webui/option...
File chrome/browser/ui/webui/options2/chromeos/internet_options_handler2.cc
(right):

http://codereview.chromium.org/10175010/diff/1/chrome/browser/ui/webui/option...
chrome/browser/ui/webui/options2/chromeos/internet_options_handler2.cc:439:
web_ui()->RegisterMessageCallback("buttonClickCallback",
On 2012/04/24 17:15:54, csilv wrote:
> optional: I think the message strings would be better as "refreshNetworks" and
> "buttonClick" (ie, omit "Callback" in the message name.) Consistent with the
> rest of this method.

Removed Callback for refreshNetworks.  References to buttonClickCallback are
quite pervasive in the javascript code.  Have a separate issue to track this as
buttonClick is too non-descriptive.  Plan to switch it to networkUpdate as a
separate CL.

[commit-bot: I haz the power, 2012-04-24 21:15:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kevers@chromium.org/10175010/10001

[commit-bot: I haz the power, 2012-04-25 00:24:52]

Change committed as 133820