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

Side by Side Diff: chrome/browser/renderer_host/resource_dispatcher_host.cc

Issue 63036: Fix a crash where the ResourceMessageFilter is deleted (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 8 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) 2006-2008 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2008 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 // See http://dev.chromium.org/developers/design-documents/multi-process-resourc e-loading 5 // See http://dev.chromium.org/developers/design-documents/multi-process-resourc e-loading
6 6
7 #include "chrome/browser/renderer_host/resource_dispatcher_host.h" 7 #include "chrome/browser/renderer_host/resource_dispatcher_host.h"
8 8
9 #include <vector> 9 #include <vector>
10 10
(...skipping 364 matching lines...) Expand 10 before | Expand all | Expand 10 after
375 } 375 }
376 376
377 if (safe_browsing_->enabled() && 377 if (safe_browsing_->enabled() &&
378 safe_browsing_->CanCheckUrl(request_data.url)) { 378 safe_browsing_->CanCheckUrl(request_data.url)) {
379 handler = new SafeBrowsingResourceHandler(handler, 379 handler = new SafeBrowsingResourceHandler(handler,
380 process_id, 380 process_id,
381 route_id, 381 route_id,
382 request_data.url, 382 request_data.url,
383 request_data.resource_type, 383 request_data.resource_type,
384 safe_browsing_, 384 safe_browsing_,
385 this); 385 this,
386 receiver_);
386 } 387 }
387 388
388 // Insert a buffered event handler before the actual one. 389 // Insert a buffered event handler before the actual one.
389 handler = new BufferedResourceHandler(handler, this, request); 390 handler = new BufferedResourceHandler(handler, this, request);
390 391
391 // Make extra info and read footer (contains request ID). 392 // Make extra info and read footer (contains request ID).
392 ExtraRequestInfo* extra_info = 393 ExtraRequestInfo* extra_info =
393 new ExtraRequestInfo(handler, 394 new ExtraRequestInfo(handler,
394 process_type, 395 process_type,
395 process_id, 396 process_id,
(...skipping 110 matching lines...) Expand 10 before | Expand all | Expand 10 after
506 true); 507 true);
507 508
508 509
509 if (safe_browsing_->enabled() && safe_browsing_->CanCheckUrl(url)) { 510 if (safe_browsing_->enabled() && safe_browsing_->CanCheckUrl(url)) {
510 handler = new SafeBrowsingResourceHandler(handler, 511 handler = new SafeBrowsingResourceHandler(handler,
511 process_id, 512 process_id,
512 route_id, 513 route_id,
513 url, 514 url,
514 ResourceType::MAIN_FRAME, 515 ResourceType::MAIN_FRAME,
515 safe_browsing_, 516 safe_browsing_,
516 this); 517 this,
518 receiver_);
517 } 519 }
518 520
519 bool known_proto = URLRequest::IsHandledURL(url); 521 bool known_proto = URLRequest::IsHandledURL(url);
520 if (!known_proto) { 522 if (!known_proto) {
521 CHECK(false); 523 CHECK(false);
522 } 524 }
523 525
524 request->set_method("GET"); 526 request->set_method("GET");
525 request->set_referrer(referrer.spec()); 527 request->set_referrer(referrer.spec());
526 request->set_context(request_context); 528 request->set_context(request_context);
(...skipping 1027 matching lines...) Expand 10 before | Expand all | Expand 10 after
1554 case ViewHostMsg_UploadProgress_ACK::ID: 1556 case ViewHostMsg_UploadProgress_ACK::ID:
1555 case ViewHostMsg_SyncLoad::ID: 1557 case ViewHostMsg_SyncLoad::ID:
1556 return true; 1558 return true;
1557 1559
1558 default: 1560 default:
1559 break; 1561 break;
1560 } 1562 }
1561 1563
1562 return false; 1564 return false;
1563 } 1565 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698