Fixing iframe jank in the local omnibox popup.

chrome/renderer/chrome_content_renderer_client_unittest.cc

Index: chrome/renderer/chrome_content_renderer_client_unittest.cc
diff --git a/chrome/renderer/chrome_content_renderer_client_unittest.cc b/chrome/renderer/chrome_content_renderer_client_unittest.cc
index b099eea5a6f62b857835b5b26ae3dbcf6e51cbd3..ec3fc4e617dd24c7164871c6a65351e24168985d 100644
--- a/chrome/renderer/chrome_content_renderer_client_unittest.cc
+++ b/chrome/renderer/chrome_content_renderer_client_unittest.cc
@@ -342,4 +342,22 @@ TEST_F(ChromeContentRendererClientTest, IsRequestOSFileHandleAllowedForURL) {
       GURL("filesystem:http://192.168.0.1/foo")));
 }
 
+TEST_F(ChromeContentRendererClientTest, SetOriginForSuggestionRequest) {
+  ChromeContentRendererClient client;
+  GURL result;
+  client.SetOriginForSuggestionRequest(
+      GURL("chrome-search://suggestion/result.js"),
+      GURL("http://sanity.example.com"), &result);
+  EXPECT_EQ(
+      GURL("chrome-search://suggestion/result.js?"
+           "origin=http://sanity.example.com"), result);
+  client.SetOriginForSuggestionRequest(
+      GURL("chrome-search://suggestion/result.js?"
+           "origin=http://good.example.com/"),
+      GURL("http://evil.example.com"), &result);
+  EXPECT_EQ(
+      GURL("chrome-search://suggestion/result.js?"
+           "origin=http://evil.example.com"), result);

[palmer, 2013/04/09 00:26:15]

What about malformed and incomplete origins, and multiple origin parameters?

[Jered, 2013/04/09 01:18:58]

Do you mean what if the top_url param is malformed or incomplete, i.e. the
webframe URL is invalid? [The entire query string is replaced, but sure, I can
add some tests for multiple params too.]

[Jered, 2013/04/09 03:24:24]

Done, I think.

+}
+
 }  // namespace chrome