Remove iron-flex-layout from cr-network

Remove iron-flex-layout from cr-network

Helps in removing iron-flex-layout/classes. For cr-elements, this is the
only dependency on those classes, so we can do away with those files
entirely by just using normal CSS.

BUG=635633
R=stevenjb@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

[michaelpg, 2016-08-10 16:13:35]

Description was changed from

==========
Remove iron-flex-layout from cr-network

Helps in removing iron-flex-layout/classes. For cr-elements, this is the
only dependency on those classes, so we can do away with those files
entirely by just using normal CSS.

BUG=635633
R=stevenjb@chromium.org
==========

to

==========
Remove iron-flex-layout from cr-network

Helps in removing iron-flex-layout/classes. For cr-elements, this is the
only dependency on those classes, so we can do away with those files
entirely by just using normal CSS.

BUG=635633
R=stevenjb@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[stevenjb, 2016-08-12 23:20:48]

lgtm

https://codereview.chromium.org/2236463005/diff/1/ui/webui/resources/cr_eleme...
File ui/webui/resources/cr_elements/network/cr_network_select.html (right):

https://codereview.chromium.org/2236463005/diff/1/ui/webui/resources/cr_eleme...
ui/webui/resources/cr_elements/network/cr_network_select.html:26: }
nit: FWIW I find all of these custom rules much less readable than the layout
classes.

That said, it sounds like there is a non trivial amount of overhead with
including the iron layout style, so for anything inside cr_elements/ it seems
reasonable to not include it.

https://codereview.chromium.org/2236463005/diff/1/ui/webui/resources/cr_eleme...
ui/webui/resources/cr_elements/network/cr_network_select.html:29: <header>
nit: I'd rather just use a div here. I don't think there is a clear consensus on
where we should use <header> vs <div>, and since we don't use it anywhere else
in settings, I would prefer not to start here.