MD Settings: inline bluetooth_dialog_shared_css.html now that it's only

MD Settings: inline bluetooth_dialog_shared_css.html now that it's only
used in 1 place.

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

Committed: https://crrev.com/f7f6b761518b0450d53a5e0db65df71a1f62855d
Cr-Commit-Position: refs/heads/master@{#393110}

[Dan Beam, 2016-05-07 03:13:24]

Description was changed from

==========
MD Settings: inline bluetooth_dialog_shared_css.html now that it's only
used in 1 place.

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

to

==========
MD Settings: inline bluetooth_dialog_shared_css.html now that it's only
used in 1 place.

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

[stevenjb, 2016-05-09 17:02:57]

https://codereview.chromium.org/1954413002/diff/1/chrome/browser/resources/se...
File
chrome/browser/resources/settings/bluetooth_page/bluetooth_device_dialog.html
(right):

https://codereview.chromium.org/1954413002/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/bluetooth_page/bluetooth_device_dialog.html:96:
</style>
If inlining CSS is the preference (requirement? I only partially recall how that
discussion went) now, we should add a note in the WebUI style guide (in sites,
or in the draft doc at least).

[Dan Beam, 2016-05-09 18:27:37]

https://codereview.chromium.org/1954413002/diff/1/chrome/browser/resources/se...
File
chrome/browser/resources/settings/bluetooth_page/bluetooth_device_dialog.html
(right):

https://codereview.chromium.org/1954413002/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/bluetooth_page/bluetooth_device_dialog.html:96:
</style>
On 2016/05/09 17:02:57, stevenjb wrote:
> If inlining CSS is the preference (requirement? I only partially recall how
that
> discussion went) now, we should add a note in the WebUI style guide (in sites,
> or in the draft doc at least).

i don't think we need to be prescriptive about unsharing things.

[stevenjb, 2016-05-11 16:41:47]

I assume we'll look into sharing CSS properties in the next pass when we use the
settings dialog header/footer, so LGTM.

[Dan Beam, 2016-05-11 23:49:37]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2016-05-11 23:49:38]

The patchset sent to the CQ was uploaded after l-g-t-m from
stevenjb@chromium.org
Link to the patchset: https://codereview.chromium.org/1954413002/#ps80001
(title: "merge")

[commit-bot: I haz the power, 2016-05-11 23:50:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1954413002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1954413002/80001

[commit-bot: I haz the power, 2016-05-11 23:54:35]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-05-11 23:55:59]

Description was changed from

==========
MD Settings: inline bluetooth_dialog_shared_css.html now that it's only
used in 1 place.

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

to

==========
MD Settings: inline bluetooth_dialog_shared_css.html now that it's only
used in 1 place.

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

Committed: https://crrev.com/f7f6b761518b0450d53a5e0db65df71a1f62855d
Cr-Commit-Position: refs/heads/master@{#393110}
==========

[commit-bot: I haz the power, 2016-05-11 23:55:59]

Patchset 5 (id:??) landed as
https://crrev.com/f7f6b761518b0450d53a5e0db65df71a1f62855d
Cr-Commit-Position: refs/heads/master@{#393110}