WebUI "Edit Search Engine" Dialog now wider and deeper and buttons bottom justified

chrome/browser/resources/edit_search_engine_dialog.css

Index: chrome/browser/resources/edit_search_engine_dialog.css
diff --git a/chrome/browser/resources/edit_search_engine_dialog.css b/chrome/browser/resources/edit_search_engine_dialog.css
index 0f89cfcb9b3056d9387051865703c5096eac5898..de4b2c12fb863ff33764559249e6b7bf9b94c785 100644
--- a/chrome/browser/resources/edit_search_engine_dialog.css
+++ b/chrome/browser/resources/edit_search_engine_dialog.css
@@ -34,12 +34,18 @@ table {
 }
 
 .action-area {
-  -webkit-box-align: center;
-  -webkit-box-orient: horizontal;
-  -webkit-box-pack: end;
-  display: -webkit-box;
+  bottom: 0;
+  padding: 12px;
+  position: absolute;

[Rick Byers, 2011/11/29 22:06:47]

I believe the use of webkit-box here was to get the two buttons to be the same
size - both big enough to fit the longer of the two text strings inside them
(regardless of language).  See the comments on the original CL here:
http://codereview.chromium.org/8118012/

It seems like we make an effort to do this elsewhere too (eg. all the buttons on
the options page - see
http://codesearch.google.com/#OAMlx_jo-ck/src/chrome/browser/resources/option...)
so I assume we want to preserve it here too.

It's also nice that using -webkit-box-pack: end avoids the need for a separate
rtl rule.

Can you not just keep all the existing stuff but add "position: fixed, bottom:
0"?

+  right: 0;
+}
+
+html[dir=rtl] .action-area {
+  bottom: 0;

[arv (Not doing code reviews), 2011/11/29 21:55:30]

no need to duplicate bottom, padding nor position.

+  left: 0;
   padding: 12px;
-  text-align: end;
+  position: absolute;
+  right: auto;
 }
 
 .button-strip {