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

Unified Diff: pdf/document_loader.h

Issue 2407683002: PDF: Check the loaded URL in sendScriptingMessage_(). (Closed)
Patch Set: Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
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_;

Powered by Google App Engine
This is Rietveld 408576698