Fix button order to OK, Cancel

Fix button order to OK, Cancel

BUG=chromium-os:29684
TEST=Manual test.


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

[James Hawkins, 2012-04-24 16:25:08]

To be clear, the expected order for CrOS is "OK/Cancel", correct?

[SeRya, 2012-04-24 20:19:52]

On 2012/04/24 16:25:08, James Hawkins wrote:
> To be clear, the expected order for CrOS is "OK/Cancel", correct?

Correct. And in all others overlays it is.

[James Hawkins, 2012-04-24 20:35:20]

lgtm

[commit-bot: I haz the power, 2012-04-25 02:38:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/serya@chromium.org/10155030/1

[commit-bot: I haz the power, 2012-04-25 05:03:29]

Change committed as 133864

[Lei Zhang, 2012-04-26 22:31:08]

Umm, in several cases, like with print preview, this flipped it for all
platforms.

On Linux, it was [cancel] [Print] and now it's [Print] [cancel].

[Evan Stade, 2012-04-26 22:51:08]

I think print preview is the only one that is wrong. The changes to the pack
extension overlay and alert overlay were correct, and the rest are chromeos
only.

http://codereview.chromium.org/10155030/diff/1/chrome/browser/resources/exten...
File chrome/browser/resources/extensions/pack_extension_overlay.html (right):

http://codereview.chromium.org/10155030/diff/1/chrome/browser/resources/exten...
chrome/browser/resources/extensions/pack_extension_overlay.html:23: <button
id="packExtensionDismiss" i18n-content="cancel"></button>
this is correct

http://codereview.chromium.org/10155030/diff/1/chrome/browser/resources/print...
File chrome/browser/resources/print_preview/print_preview.html (right):

http://codereview.chromium.org/10155030/diff/1/chrome/browser/resources/print...
chrome/browser/resources/print_preview/print_preview.html:28: <button
id="cancel-button" i18n-content="cancelButton"></button>
this is wrong --- if the css rule in print_preview.css is not working as
expected, it should be fixed.