Fixing iframe jank in the local omnibox popup.

chrome/browser/ui/webui/theme_source.cc

Index: chrome/browser/ui/webui/theme_source.cc
diff --git a/chrome/browser/ui/webui/theme_source.cc b/chrome/browser/ui/webui/theme_source.cc
index 26f76db793d0250e90d2ded6b059773576856a7f..0a87c0809fe96a08873cbdcb5cb3cd9bc9dec38a 100644
--- a/chrome/browser/ui/webui/theme_source.cc
+++ b/chrome/browser/ui/webui/theme_source.cc
@@ -56,7 +56,8 @@ std::string ThemeSource::GetSource() const {
 
 void ThemeSource::StartDataRequest(
     const std::string& path,
-    bool is_incognito,
+    int render_process_id,
+    int render_view_id,
     const content::URLDataSource::GotDataCallback& callback) {
   // Default scale factor if not specified.
   ui::ScaleFactor scale_factor;
@@ -68,8 +69,6 @@ void ThemeSource::StartDataRequest(
   if (uncached_path == kNewTabCSSPath ||
       uncached_path == kNewIncognitoTabCSSPath) {
     DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
-    DCHECK((uncached_path == kNewTabCSSPath && !is_incognito) ||
-           (uncached_path == kNewIncognitoTabCSSPath && is_incognito));

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

Are we sure it's safe to remove this DCHECK?

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

Yes. The owners agreed this was ok in https://codereview.chromium.org/14039004/
of which this is a patch.

 
     callback.Run(css_bytes_);
     return;