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

Side by Side Diff: ppapi/native_client/src/trusted/plugin/pnacl_coordinator.cc

Issue 181933005: Pepper: Simplify FileDownloader::url(). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "ppapi/native_client/src/trusted/plugin/pnacl_coordinator.h" 5 #include "ppapi/native_client/src/trusted/plugin/pnacl_coordinator.h"
6 6
7 #include <utility> 7 #include <utility>
8 #include <vector> 8 #include <vector>
9 9
10 #include "native_client/src/include/checked_cast.h" 10 #include "native_client/src/include/checked_cast.h"
(...skipping 474 matching lines...) Expand 10 before | Expand all | Expand 10 after
485 nacl::string headers = streaming_downloader_->GetResponseHeaders(); 485 nacl::string headers = streaming_downloader_->GetResponseHeaders();
486 NaClHttpResponseHeaders parser; 486 NaClHttpResponseHeaders parser;
487 parser.Parse(headers); 487 parser.Parse(headers);
488 488
489 temp_nexe_file_.reset(new TempFile(plugin_)); 489 temp_nexe_file_.reset(new TempFile(plugin_));
490 pp::CompletionCallback cb = 490 pp::CompletionCallback cb =
491 callback_factory_.NewCallback(&PnaclCoordinator::NexeFdDidOpen); 491 callback_factory_.NewCallback(&PnaclCoordinator::NexeFdDidOpen);
492 int32_t nexe_fd_err = 492 int32_t nexe_fd_err =
493 plugin_->nacl_interface()->GetNexeFd( 493 plugin_->nacl_interface()->GetNexeFd(
494 plugin_->pp_instance(), 494 plugin_->pp_instance(),
495 streaming_downloader_->url().c_str(), 495 streaming_downloader_->full_url().c_str(),
496 // TODO(dschuff): Get this value from the pnacl json file after it 496 // TODO(dschuff): Get this value from the pnacl json file after it
497 // rolls in from NaCl. 497 // rolls in from NaCl.
498 1, 498 1,
499 pnacl_options_.opt_level(), 499 pnacl_options_.opt_level(),
500 parser.GetHeader("last-modified").c_str(), 500 parser.GetHeader("last-modified").c_str(),
501 parser.GetHeader("etag").c_str(), 501 parser.GetHeader("etag").c_str(),
502 PP_FromBool(parser.CacheControlNoStore()), 502 PP_FromBool(parser.CacheControlNoStore()),
503 GetSandboxISA(), 503 GetSandboxISA(),
504 "", // No extra compile flags yet. 504 "", // No extra compile flags yet.
505 &is_cache_hit_, 505 &is_cache_hit_,
(...skipping 176 matching lines...) Expand 10 before | Expand all | Expand 10 after
682 temp_nexe_file_.get(), 682 temp_nexe_file_.get(),
683 invalid_desc_wrapper_.get(), 683 invalid_desc_wrapper_.get(),
684 &error_info_, 684 &error_info_,
685 resources_.get(), 685 resources_.get(),
686 &pnacl_options_, 686 &pnacl_options_,
687 this, 687 this,
688 plugin_); 688 plugin_);
689 } 689 }
690 690
691 } // namespace plugin 691 } // namespace plugin
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698