Fixing iframe jank in the local omnibox popup.

chrome/browser/resources/omnibox_result.js

Index: chrome/browser/resources/omnibox_result.js
diff --git a/chrome/browser/resources/omnibox_result.js b/chrome/browser/resources/omnibox_result.js
new file mode 100644
index 0000000000000000000000000000000000000000..cff0f2bf7a1d32d5367ff8c643a40a9fe98b78a6
--- /dev/null
+++ b/chrome/browser/resources/omnibox_result.js
@@ -0,0 +1,20 @@
+/**

[Dan Beam, 2013/04/05 01:07:37]

does this need a copyright?

[Jered, 2013/04/05 15:30:23]

Done.

+ * @fileoverview Click handler for omnibox results.
+ */
+
+/**
+ * The origin of the embedding page.
+ * Must be in double quotes for proper escaping.

[Dan Beam, 2013/04/05 01:07:37]

what must be in double quotes? please clarify

[Jered, 2013/04/05 15:30:23]

Done.

+ * @type {string}
+ * @const
+ */
+var EMBEDDER_ORIGIN = "%s";
+
+function handleClick(event) {
+  var clickMessage = {'click': event.button};
+  top.postMessage(clickMessage, EMBEDDER_ORIGIN);

[Dan Beam, 2013/04/05 01:07:37]

top.postMessage({click: event.button}, EMBEDDER_ORIGIN);

[Jered, 2013/04/05 15:30:23]

Done.

+}
+
+document.addEventListener('DOMContentLoaded', function() {

[Dan Beam, 2013/04/05 01:07:37]

why do you need to wait until DOMContentLoaded?

[Jered, 2013/04/05 15:30:23]

I don't. Done.

+  window.addEventListener('click', handleClick, false);
+});

[Dan Beam, 2013/04/05 01:07:37]

why is this all in the global namespace?

[Jered, 2013/04/05 15:30:23]

Where would be better? (This script will be the only script alive in an iframe.)