Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(146)

Side by Side Diff: chrome/browser/extensions/webstore_install_helper.cc

Issue 9565003: Merge 124282 - Disable storing cookies from URLFetchers that run with a profile's cookie jar. (Closed) Base URL: svn://svn.chromium.org/chrome/branches/963/src/
Patch Set: Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/extensions/webstore_install_helper.h" 5 #include "chrome/browser/extensions/webstore_install_helper.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/values.h" 10 #include "base/values.h"
11 #include "chrome/common/chrome_utility_messages.h" 11 #include "chrome/common/chrome_utility_messages.h"
12 #include "content/public/browser/browser_thread.h" 12 #include "content/public/browser/browser_thread.h"
13 #include "content/public/common/url_fetcher.h" 13 #include "content/public/common/url_fetcher.h"
14 #include "net/base/load_flags.h"
14 #include "net/url_request/url_request_context_getter.h" 15 #include "net/url_request/url_request_context_getter.h"
15 #include "net/url_request/url_request_status.h" 16 #include "net/url_request/url_request_status.h"
16 17
17 using content::BrowserThread; 18 using content::BrowserThread;
18 19
19 namespace { 20 namespace {
20 21
21 const char kImageDecodeError[] = "Image decode failed"; 22 const char kImageDecodeError[] = "Image decode failed";
22 23
23 } // namespace 24 } // namespace
(...skipping 27 matching lines...) Expand all
51 BrowserThread::PostTask( 52 BrowserThread::PostTask(
52 BrowserThread::IO, 53 BrowserThread::IO,
53 FROM_HERE, 54 FROM_HERE,
54 base::Bind(&WebstoreInstallHelper::StartWorkOnIOThread, this)); 55 base::Bind(&WebstoreInstallHelper::StartWorkOnIOThread, this));
55 56
56 if (!icon_url_.is_empty()) { 57 if (!icon_url_.is_empty()) {
57 CHECK(context_getter_); 58 CHECK(context_getter_);
58 url_fetcher_.reset(content::URLFetcher::Create( 59 url_fetcher_.reset(content::URLFetcher::Create(
59 icon_url_, content::URLFetcher::GET, this)); 60 icon_url_, content::URLFetcher::GET, this));
60 url_fetcher_->SetRequestContext(context_getter_); 61 url_fetcher_->SetRequestContext(context_getter_);
62 url_fetcher_->SetLoadFlags(net::LOAD_DO_NOT_SAVE_COOKIES);
61 63
62 url_fetcher_->Start(); 64 url_fetcher_->Start();
63 // We'll get called back in OnURLFetchComplete. 65 // We'll get called back in OnURLFetchComplete.
64 } 66 }
65 } 67 }
66 68
67 void WebstoreInstallHelper::StartWorkOnIOThread() { 69 void WebstoreInstallHelper::StartWorkOnIOThread() {
68 CHECK(BrowserThread::CurrentlyOn(BrowserThread::IO)); 70 CHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
69 utility_host_ = 71 utility_host_ =
70 (new UtilityProcessHost(this, BrowserThread::IO))->AsWeakPtr(); 72 (new UtilityProcessHost(this, BrowserThread::IO))->AsWeakPtr();
(...skipping 111 matching lines...) Expand 10 before | Expand all | Expand 10 after
182 base::Bind(&WebstoreInstallHelper::ReportResultFromUIThread, this)); 184 base::Bind(&WebstoreInstallHelper::ReportResultFromUIThread, this));
183 } 185 }
184 186
185 void WebstoreInstallHelper::ReportResultFromUIThread() { 187 void WebstoreInstallHelper::ReportResultFromUIThread() {
186 CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 188 CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
187 if (error_.empty() && parsed_manifest_.get()) 189 if (error_.empty() && parsed_manifest_.get())
188 delegate_->OnWebstoreParseSuccess(id_, icon_, parsed_manifest_.release()); 190 delegate_->OnWebstoreParseSuccess(id_, icon_, parsed_manifest_.release());
189 else 191 else
190 delegate_->OnWebstoreParseFailure(id_, parse_error_, error_); 192 delegate_->OnWebstoreParseFailure(id_, parse_error_, error_);
191 } 193 }
OLDNEW
« no previous file with comments | « chrome/browser/autofill/autofill_download.cc ('k') | chrome/browser/importer/toolbar_importer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698