Fixing iframe jank in the local omnibox popup.

content/browser/webui/web_ui_data_source_unittest.cc

Index: content/browser/webui/web_ui_data_source_unittest.cc
diff --git a/content/browser/webui/web_ui_data_source_unittest.cc b/content/browser/webui/web_ui_data_source_unittest.cc
index bece8d0fcddf4707bc923d6bb142639c9d9c9698..1a5efb795edccae410918a11126b3cfa3c92a684 100644
--- a/content/browser/webui/web_ui_data_source_unittest.cc
+++ b/content/browser/webui/web_ui_data_source_unittest.cc
@@ -59,7 +59,7 @@ class WebUIDataSourceTest : public testing::Test {
   void StartDataRequest(const std::string& path) {
      source_->StartDataRequest(
         path,
-        false,
+        0, 0,

[palmer, 2013/04/22 19:38:37]

Elsewhere, you used -1 to indicate no particular IDs. Which is right?

[Jered, 2013/04/22 23:02:58]

It's a no-op in all these test cases because it's ignored. I just haphazardly
picked 0. Would -1 be clearer?

[palmer, 2013/04/23 00:34:24]

I only asked because you used -1 elsewhere, and it seems better to have just one
"this value is meaningless" value. It's not a huge deal.

         base::Bind(&WebUIDataSourceTest::SendResult,
         base::Unretained(this)));
   }