Ensure that isolated apps use the right cookies for media requests.

chrome/browser/browsing_data/browsing_data_remover.cc

Index: chrome/browser/browsing_data/browsing_data_remover.cc
diff --git a/chrome/browser/browsing_data/browsing_data_remover.cc b/chrome/browser/browsing_data/browsing_data_remover.cc
index 576f3d1eb0484cae19aef63f60054ec158f6bf2e..cb7c5088a3ffe291331316781baf2da90231d89f 100644
--- a/chrome/browser/browsing_data/browsing_data_remover.cc
+++ b/chrome/browser/browsing_data/browsing_data_remover.cc
@@ -112,7 +112,7 @@ BrowsingDataRemover::BrowsingDataRemover(Profile* profile,
       next_cache_state_(STATE_NONE),
       cache_(NULL),
       main_context_getter_(profile->GetRequestContext()),
-      media_context_getter_(profile->GetRequestContextForMedia()),
+      media_context_getter_(profile->GetRequestContextForMedia(-1)),

[awong, 2012/08/17 18:23:16]

Ick...on the magic -1 everywhere.

I had been using ChildProcessHostImpl::kInvalidChildProcessId.  Maybe that could
be made public?

[Charlie Reis, 2012/08/17 21:04:38]

I've switched it to use a GetDefaultRequestContextForMedia() for the time being.
 We'll need to revisit all of this anyway for the transition to
StoragePartition.

       deauthorize_content_licenses_request_id_(0),
       waiting_for_clear_cache_(false),
       waiting_for_clear_nacl_cache_(false),
@@ -148,7 +148,7 @@ BrowsingDataRemover::BrowsingDataRemover(Profile* profile,
       next_cache_state_(STATE_NONE),
       cache_(NULL),
       main_context_getter_(profile->GetRequestContext()),
-      media_context_getter_(profile->GetRequestContextForMedia()),
+      media_context_getter_(profile->GetRequestContextForMedia(-1)),
       deauthorize_content_licenses_request_id_(0),
       waiting_for_clear_cache_(false),
       waiting_for_clear_nacl_cache_(false),