Unframe chrome://extensions as it's the last (and only) uber item

chrome/browser/resources/extensions/extension_list.js

Index: chrome/browser/resources/extensions/extension_list.js
diff --git a/chrome/browser/resources/extensions/extension_list.js b/chrome/browser/resources/extensions/extension_list.js
index 235c75809788a16f6dc75e27e59c8ee0566614f2..5582d587065f5b0213acbdab79d5b9088a3d15e1 100644
--- a/chrome/browser/resources/extensions/extension_list.js
+++ b/chrome/browser/resources/extensions/extension_list.js
@@ -984,7 +984,7 @@ cr.define('extensions', function() {
       // Add the options query string. Corner case: the 'options' query string
       // will clobber the 'id' query string if the options link is clicked when
       // 'id' is in the URL, or if both query strings are in the URL.
-      uber.replaceState({}, '?options=' + extensionId);
+      window.history.replaceState({}, '', '/?options=' + extensionId);

[Devlin, 2017/06/12 14:17:18]

why window.history instead of history?  (I can't find any style rules here, but
it seems odd)

[Dan Beam, 2017/06/12 23:13:12]

it's because of the way it's externed:
https://github.com/google/closure-compiler/blob/41263520e01176aa03258b3868a23...

[Devlin, 2017/06/13 18:04:21]

Ah, interesting...

[Dan Beam, 2017/06/13 18:15:16]

Acknowledged.

 
       var overlay = extensions.ExtensionOptionsOverlay.getInstance();
       var shownCallback = function() {
@@ -1003,7 +1003,7 @@ cr.define('extensions', function() {
         $('overlay').removeEventListener('cancelOverlay', f);
 
         // Remove the options query string.
-        uber.replaceState({}, '');
+        window.history.replaceState({}, '', '/');
       });
 
       // TODO(dbeam): why do we need to focus <extensionoptions> before and