Convert chrome://extensions to a settings page within the options pages.

chrome/browser/resources/options/extension_settings.html

Index: chrome/browser/resources/options/extension_settings.html
===================================================================
--- chrome/browser/resources/options/extension_settings.html	(revision 0)
+++ chrome/browser/resources/options/extension_settings.html	(revision 0)
@@ -0,0 +1,38 @@
+<div id="extension-settings" class="page" hidden>
+  <h1 i18n-content="extensionSettingsTitle"></h1>
+  <div id="dev-toggle">
+    <input id="toggle-dev-on" type="checkbox" value="off"></input>
+    <label title="Toggle developer mode" for="toggle-dev-on"

[csilv, 2011/09/01 19:52:53]

The title should be localized.  Instead of this attribute, use:
  i18n-values="title:toggleDeveloperMode"

[Finnur, 2011/09/02 13:58:47]

This tooltip is kind of useless and obvious. Removed.

On 2011/09/01 19:52:53, csilv wrote:

+           i18n-content="developerMode" />
+  </div>
+  <div class="displaytable">
+    <div id="dev" class="dev-closed">
+      <table id="dev-table" width="100%">
+        <tr>
+          <td>
+            <button id="load-unpacked">Load unpacked extension...</button>

[csilv, 2011/09/01 19:52:53]

Use i18n-content to localize strings, here and below.

[Finnur, 2011/09/02 13:58:47]

Ah, good catch!

On 2011/09/01 19:52:53, csilv wrote:

+            <button id="pack-extension">Pack extension...</button>
+          </td>
+          <td align="right">
+            <button id="update-extensions-now">Update extension now</button>
+          </td>
+        </tr>
+      </table>
+    </div>
+    <section class="content">
+      <div class="extension-settings">
+        <list class="big-list" id="extension-settings-list"></list>
+      </div>
+    </section>
+  </div>
+  <section>
+    <div><strong id="no-extensions"

[csilv, 2011/09/01 19:52:53]

set the 'hidden' attribute rather than the style tag.

[Finnur, 2011/09/02 13:58:47]

Done.

+                 i18n-content="noExtensions"
+                 style="display: none"></strong></div>
+    <div id="suggest-gallery" style="display: none"></div>
+    <div id="get-more-extensions-container">
+      <image src="chrome://theme/IDR_WEBSTORE_ICON_32" />

[csilv, 2011/09/01 19:52:53]

don't need the trailing slash for image tags w/ HTML5.

[Finnur, 2011/09/02 13:58:47]

Done.

[arv (Not doing code reviews), 2011/09/06 20:55:47]

<img>

<image> is an IE compat quirk

+      <div id="get-more-extensions"></div>
+    </div>
+  </section>
+</div>
Property changes on: chrome\browser\resources\options\extension_settings.html
___________________________________________________________________
Added: svn:eol-style
   + LF