Stub in class for layout of network controls on the main settings page.

Stub in class for layout of network controls on the main settings page.

BUG=chromium:110749, chromium:112293
TEST=


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

[kevers, 2012-02-02 22:29:18]

Hi Chris,

Can you have a look at this CL, which provides classes for grouping networks by
type for display on the main settings page.

[csilv, 2012-02-03 19:54:30]

lgtm

http://codereview.chromium.org/9317064/diff/1/chrome/browser/resources/option...
File chrome/browser/resources/options2/chromeos/network_list.js (right):

http://codereview.chromium.org/9317064/diff/1/chrome/browser/resources/option...
chrome/browser/resources/options2/chromeos/network_list.js:167:
NetworkList.refreshNetworkData = function (data) {
nit: remove space before paren

[kevers, 2012-02-06 17:24:43]

http://codereview.chromium.org/9317064/diff/1/chrome/browser/resources/option...
File chrome/browser/resources/options2/chromeos/network_list.js (right):

http://codereview.chromium.org/9317064/diff/1/chrome/browser/resources/option...
chrome/browser/resources/options2/chromeos/network_list.js:167:
NetworkList.refreshNetworkData = function (data) {
On 2012/02/03 19:54:30, csilv wrote:
> nit: remove space before paren

Done.

[commit-bot: I haz the power, 2012-02-06 17:25:31]

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

[commit-bot: I haz the power, 2012-02-06 19:58:34]

Try job failure for 9317064-3001 (retry) on win_rel for step "browser_tests".
It's a second try, previously, step "browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2012-02-06 20:11:03]

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

[commit-bot: I haz the power, 2012-02-06 23:42:02]

Try job failure for 9317064-3001 (retry) (retry) on mac_rel for step
"browser_tests".
It's a second try, previously, step "browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-02-07 00:44:25]

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

[commit-bot: I haz the power, 2012-02-07 02:53:39]

Change committed as 120695