Kill DownloadItem::IsOtr()

chrome/browser/automation/automation_provider.h

Index: chrome/browser/automation/automation_provider.h
diff --git a/chrome/browser/automation/automation_provider.h b/chrome/browser/automation/automation_provider.h
index b7de6515be72327d2ef037161a4233ec1dd715bd..02ae66becdec9b25ed49d67a7c12475c41b583cf 100644
--- a/chrome/browser/automation/automation_provider.h
+++ b/chrome/browser/automation/automation_provider.h
@@ -156,7 +156,8 @@ class AutomationProvider
   // Get the DictionaryValue equivalent for a download item. Caller owns the
   // DictionaryValue.
   base::DictionaryValue* GetDictionaryFromDownloadItem(
-      const content::DownloadItem* download);
+      const content::DownloadItem* download,
+      Profile* profile);

[Randy Smith (Not in Mondays), 2012/08/02 17:52:17]

If you did this in response to my earlier comments, I'd like to apologize and
repent.  I think the code will be simpler (and lower level objects won't need to
know about Profiles) if you just pass around a boolean as to whether the
download item is incognito or not.

[benjhayden, 2012/08/13 15:08:09]

Done.

 
  protected:
   friend struct content::BrowserThread::DeleteOnThread<