Browser test for LoadDataWithBaseURL reload

content/public/renderer/document_state.h

Index: content/public/renderer/document_state.h
diff --git a/content/public/renderer/document_state.h b/content/public/renderer/document_state.h
index b24c9f17c153631678febff1b9209bbfb7f865f7..957053222c132dcaf5cab415cc4809c7ec51f9cb 100644
--- a/content/public/renderer/document_state.h
+++ b/content/public/renderer/document_state.h
@@ -14,6 +14,7 @@
 #include "content/common/content_export.h"
 #include "net/http/http_response_info.h"
 #include "third_party/WebKit/public/web/WebDataSource.h"
+#include "url/gurl.h"
 
 namespace content {
 
@@ -181,6 +182,19 @@ class CONTENT_EXPORT DocumentState
   }
   bool was_after_preconnect_request() { return was_after_preconnect_request_; }
 
+  void set_was_load_data_url_with_base_url_request(bool value) {

[Charlie Reis, 2015/11/20 21:30:12]

nit: s/data_url/data/

(Since the API is LoadDataWithBaseURL.)

[boliu, 2015/11/23 22:32:25]

Done.

+    was_load_data_url_with_base_url_request_ = value;
+  }
+  bool was_load_data_url_with_base_url_request() const {

[Charlie Reis, 2015/11/20 21:30:12]

Same.

[boliu, 2015/11/23 22:32:25]

Done.

+    return was_load_data_url_with_base_url_request_;
+  }
+  const GURL& data_url() const {
+    return data_url_;
+  }
+  void set_data_url(const GURL& data_url) {
+    data_url_ = data_url;
+  }
+
   // Record the nature of this load, for use when histogramming page load times.
   LoadType load_type() const { return load_type_; }
   void set_load_type(LoadType load_type) { load_type_ = load_type; }
@@ -215,6 +229,8 @@ class CONTENT_EXPORT DocumentState
   bool was_prefetcher_;
   bool was_referred_by_prefetcher_;
   bool was_after_preconnect_request_;
+  bool was_load_data_url_with_base_url_request_;

[Charlie Reis, 2015/11/20 21:30:12]

Same.

Also, these seem like they belong on NavigationState instead of DocumentState. 
And that already has CommonParams with url, base_url_for_data_url, and
history_url_for_data_url, so maybe we don't need to store anything extra after
all?

[boliu, 2015/11/23 22:32:25]

Did not make this change. As noted, this breaks in-page navigations, where
NavigationState is completely recreated from scratch.

+  GURL data_url_;
 
   LoadType load_type_;