Extension Commands: Make sure we can open the Configure Commands overlay from the install bubble.

Extension Commands: Make sure we can open the Configure Commands overlay from the install bubble.

When an extension with an Extension Command is installed, we show an installation bubble. We want to add to that bubble a "Configure shortcut" link that takes you directly to the Extensions page with the Configure Command overlay open. We have infrastructure for doing this in the Settings page, but that infrastructure isn't there for the Extensions page. This buys us time, allowing us to complete the installation bubble link.

BUG=145951
TEST=None
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=154752

[Finnur, 2012-08-31 16:29:31]

http://codereview.chromium.org/10911032/diff/3002/chrome/browser/resources/ex...
File chrome/browser/resources/extensions/extensions.js (right):

http://codereview.chromium.org/10911032/diff/3002/chrome/browser/resources/ex...
chrome/browser/resources/extensions/extensions.js:125: }
I opted for a similar but simpler scheme of opening overlays (than the one we
have on the options page). I looked into incorporating the same scheme we have
for the options page, but the framework for doing this on the options page isn't
there on the extensions page and it looks like a lot of work to figure out all
the things that need to be incorporated and factor them out so they can be
shared.

Some things to bear in mind:

The only purpose of this it to support clicking a [Configure Shortcut] link
inside a bubble within the Chrome UI (extension install bubble).

We don't have another overlay that we want to be addressable through a URL at
present time.

This doesn't need to be user discoverable or even usable manually, although
adding configureCommands by hand to the URL does the right thing.

[Finnur, 2012-09-03 14:18:15]

jhawkins: ping.

BTW, here is what I intend to use this change for:
http://codereview.chromium.org/10910064/
(just an FYI).

[Finnur, 2012-09-04 10:49:47]

Ah, I had forgotten about labor day. Anyway, please take a look when you get in.

[James Hawkins, 2012-09-04 14:27:59]

lgtm