Make the body not scrollable and make a scrollable container div to host #extension-settings.

chrome/browser/resources/extensions/extensions.html

Index: chrome/browser/resources/extensions/extensions.html
diff --git a/chrome/browser/resources/extensions/extensions.html b/chrome/browser/resources/extensions/extensions.html
index b6ff0f08a72afae548a048b4220c716db0172750..42df1181b57f469fc1698b8c57d995350e37edc6 100644
--- a/chrome/browser/resources/extensions/extensions.html
+++ b/chrome/browser/resources/extensions/extensions.html
@@ -54,60 +54,62 @@
   </div>
 </div>
 
-<div class="page" id="extension-settings">
-  <header id="page-header"><h1 i18n-content="extensionSettings"></h1>
-    <div id="header-controls" class="header-extras">
-      <div id="dev-toggle" class="checkbox">
-        <label>
-          <input id="toggle-dev-on" type="checkbox">
-          <span i18n-content="extensionSettingsDeveloperMode">
-        </label>
+<div id="page-container">
+  <div class="page" id="extension-settings">
+    <header id="page-header"><h1 i18n-content="extensionSettings"></h1>
+      <div id="header-controls" class="header-extras">
+        <div id="dev-toggle" class="checkbox">
+          <label>
+            <input id="toggle-dev-on" type="checkbox">

[miket_OOO, 2013/08/15 15:36:20]

I think this needs a closing tag

[xiyuan, 2013/08/15 16:24:54]

This should be fine in HTML. It only needs to be closed for XHTML.

+            <span i18n-content="extensionSettingsDeveloperMode">

[miket_OOO, 2013/08/15 15:36:20]

missing </span>

(please check the rest and make sure the markup validates)

[zhchbin, 2013/08/16 01:49:27]

Done. I have validated it with W3C Markup Validation Service.

(I just made indented these line, seems like a problem left over by history.)

+          </label>
+        </div>
       </div>
-    </div>
-    <div class="page-banner profile-is-managed-banner">
-      <div class="page-banner-gradient">
-        <span class="page-banner-text"
-            i18n-content="extensionSettingsManagedMode"></span>
+      <div class="page-banner profile-is-managed-banner">
+        <div class="page-banner-gradient">
+          <span class="page-banner-text"
+              i18n-content="extensionSettingsManagedMode"></span>
+        </div>
       </div>
-    </div>
-  </header>
-  <div id="dev-controls" hidden>
-    <div id="apps-dev-tool-banner">
-      <div class="apps-dev-tool-banner-text"
-          i18n-content="extensionSettingsUseAppsDevTools"></div>
-      <button id="open-apps-dev-tools"
-          i18n-content="extensionSettingsOpenAppsDevTools"></button>
-    </div>
-    <div class="buttons-container">
-      <button id="load-unpacked"
-          i18n-content="extensionSettingsLoadUnpackedButton"></button>
-      <button id="pack-extension"
-          i18n-content="extensionSettingsPackButton"></button>
+    </header>
+    <div id="dev-controls" hidden>
+      <div id="apps-dev-tool-banner">
+        <div class="apps-dev-tool-banner-text"
+            i18n-content="extensionSettingsUseAppsDevTools"></div>
+        <button id="open-apps-dev-tools"
+            i18n-content="extensionSettingsOpenAppsDevTools"></button>
+      </div>
+      <div class="buttons-container">
+        <button id="load-unpacked"
+            i18n-content="extensionSettingsLoadUnpackedButton"></button>
+        <button id="pack-extension"
+            i18n-content="extensionSettingsPackButton"></button>
 <if expr="pp_ifdef('chromeos')">
-      <button id="add-kiosk-app"
-          i18n-content="addKioskAppButton" hidden></button>
+        <button id="add-kiosk-app"
+            i18n-content="addKioskAppButton" hidden></button>
 </if>
-      <div id="dev-controls-spacer"></div>
-      <button id="update-extensions-now"
-          i18n-content="extensionSettingsUpdateButton"></button>
+        <div id="dev-controls-spacer"></div>
+        <button id="update-extensions-now"
+            i18n-content="extensionSettingsUpdateButton"></button>
+      </div>
+    </div>
+    <div id="extension-settings-list" class="empty-extension-list"></div>
+    <div id="no-extensions">
+      <span id="no-extensions-message"
+          i18n-content="extensionSettingsNoExtensions"></span>
+      <span id="suggest-gallery" class="more-extensions-link"
+          i18n-values=".innerHTML:extensionSettingsSuggestGallery">
+      </span>
+    </div>
+    <div id="footer-section">
+      <a target="_blank" class="more-extensions-link"
+          i18n-values="href:extensionSettingsGetMoreExtensionsUrl"
+          i18n-content="extensionSettingsGetMoreExtensions"></a>
+      <a target="_blank" hidden
+          class="extension-commands-config"
+          i18n-content="extensionSettingsCommandsLink"
+          href="#"></a>
     </div>
-  </div>
-  <div id="extension-settings-list" class="empty-extension-list"></div>
-  <div id="no-extensions">
-    <span id="no-extensions-message"
-        i18n-content="extensionSettingsNoExtensions"></span>
-    <span id="suggest-gallery" class="more-extensions-link"
-        i18n-values=".innerHTML:extensionSettingsSuggestGallery">
-    </span>
-  </div>
-  <div id="footer-section">
-    <a target="_blank" class="more-extensions-link"
-        i18n-values="href:extensionSettingsGetMoreExtensionsUrl"
-        i18n-content="extensionSettingsGetMoreExtensions"></a>
-    <a target="_blank" hidden
-        class="extension-commands-config"
-        i18n-content="extensionSettingsCommandsLink"
-        href="#"></a>
   </div>
 </div>