Sync UI: Fix up the HTML by moving divs outside of the label.

Sync UI: Fix up the HTML by moving divs outside of the label.

Also fix style issues while I'm here.

BUG=none
TEST=none
R=dbeam


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

[Dan Beam, 2012-04-23 21:25:10]

http://codereview.chromium.org/10186011/diff/1/chrome/browser/resources/sync_...
File chrome/browser/resources/sync_setup_overlay.js (right):

http://codereview.chromium.org/10186011/diff/1/chrome/browser/resources/sync_...
chrome/browser/resources/sync_setup_overlay.js:290: var datatypeSelect =
$('sync-select-datatypes');
where is util.js (the file that defines $) being included? I didn't see it in
this CL and I assume it's already there, but there have been regressions when
this isn't added as well...

[James Hawkins, 2012-04-23 21:32:04]

http://codereview.chromium.org/10186011/diff/1/chrome/browser/resources/sync_...
File chrome/browser/resources/sync_setup_overlay.js (right):

http://codereview.chromium.org/10186011/diff/1/chrome/browser/resources/sync_...
chrome/browser/resources/sync_setup_overlay.js:290: var datatypeSelect =
$('sync-select-datatypes');
On 2012/04/23 21:25:10, Dan Beam wrote:
> where is util.js (the file that defines $) being included? I didn't see it in
> this CL and I assume it's already there, but there have been regressions when
> this isn't added as well...

Done.

[James Hawkins, 2012-04-23 22:58:51]

peetal

[Dan Beam, 2012-04-23 22:59:49]

lgtm w/offline debug sess

[commit-bot: I haz the power, 2012-04-24 00:12:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jhawkins@chromium.org/10186011/8001

[commit-bot: I haz the power, 2012-04-24 02:00:10]

Change committed as 133599