 Chromium Code Reviews
 Chromium Code Reviews Issue 2407683002:
  PDF: Check the loaded URL in sendScriptingMessage_().  (Closed)
    
  
    Issue 2407683002:
  PDF: Check the loaded URL in sendScriptingMessage_().  (Closed) 
  | Index: pdf/document_loader.h | 
| diff --git a/pdf/document_loader.h b/pdf/document_loader.h | 
| index cf85e99e5a1add35f6ef865815f2581805f31a28..c68c81ade15917c67342b81f0da789aa81d9dd32 100644 | 
| --- a/pdf/document_loader.h | 
| +++ b/pdf/document_loader.h | 
| @@ -29,6 +29,8 @@ class DocumentLoader { | 
| virtual pp::Instance* GetPluginInstance() = 0; | 
| // Creates new URLLoader based on client settings. | 
| virtual pp::URLLoader CreateURLLoader() = 0; | 
| + // Notification called when the actual loaded URL is available. | 
| + virtual void OnGotActualURL(const std::string& url) = 0; | 
| // Notification called when partial information about document is available. | 
| // Only called for urls that returns full content size and supports byte | 
| // range requests. | 
| @@ -103,6 +105,7 @@ class DocumentLoader { | 
| Client* const client_; | 
| std::string url_; | 
| + std::string actual_url_; | 
| 
Tom Sepez
2016/10/11 16:10:47
nit: maybe a comment that explains how this differ
 | 
| pp::URLLoader loader_; | 
| pp::CompletionCallbackFactory<DocumentLoader> loader_factory_; | 
| ChunkStream chunk_stream_; |