Add a help bubble pointing to the omnibox when installing an extension with

chrome/browser/extensions/extension_install_ui.cc

Index: chrome/browser/extensions/extension_install_ui.cc
diff --git a/chrome/browser/extensions/extension_install_ui.cc b/chrome/browser/extensions/extension_install_ui.cc
index 3c5f8199f400051814806053e616ab5456d445c2..2f1efd5039644cac91196e62f75da7d02cef0f25 100644
--- a/chrome/browser/extensions/extension_install_ui.cc
+++ b/chrome/browser/extensions/extension_install_ui.cc
@@ -155,6 +155,7 @@ void ExtensionInstallUI::OnInstallSuccess(const Extension* extension) {
   // Note that browser actions don't appear in incognito mode initially,
   // so fall back to the generic case.
   if ((extension->browser_action() && !browser->profile()->IsOffTheRecord()) ||
+      !extension->omnibox_keyword().empty() ||
       (extension->page_action() &&
       !extension->page_action()->default_icon_path().empty())) {

[Aaron Boodman, 2010/11/11 02:55:30]

Nit: this line needs to be indented one space.

[Matt Perry, 2010/11/12 07:20:25]

Are you sure? I don't think I've ever seen a multiply-parenthesized expression
formatted that way.

[Aaron Boodman, 2010/11/12 19:04:06]

The expression "!extension->page_action()->default_icon_path().empty()" is a
child of the left paren above the exclamation point. Typically don't we do:

if (very_long_expression &&
    other_long_expression) {

}

?

[Matt Perry, 2010/11/12 23:08:18]

Yes, but I've only seen it stop there. It looks weird to me to do:

if (expr1 &&
    (expr2 &&
     expr3) &&
    expr4) {
}

but there's nothing in the style guide either way.

     ExtensionInstalledBubbleCocoa::ShowExtensionInstalledBubble(