Fix for possible crashes due to pointer value being null

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 5dc54433f82ffb69665feb16dc69d19390c9e20e..91c896de53b393c5a9cd54b1f84e40de1ba7292e 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -3772,7 +3772,7 @@ void RenderFrameImpl::SendDidCommitProvisionalLoad(
     }
 
     base::string16 method = request.httpMethod();
-    if (EqualsASCII(method, "POST")) {
+    if (entry && EqualsASCII(method, "POST")) {
       params.is_post = true;

[Charlie Reis, 2015/02/17 21:26:59]

Good catch.  I don't think we should skip this line based on the entry being
null, though.  It's still important to mark params.is_post = true.

Looking at ExtractPostId, we should probably set params.post_id to -1 if there's
no entry.  In fact, we should probably change ExtractPostId to take in the entry
and let it handle the null case, since that's what it used to do with
WebHistoryItem before @japhet's change in
https://codereview.chromium.org/248013003.

[Sunil Ratnu, 2015/02/18 09:04:45]

Done.

       params.post_id = ExtractPostId(entry->root());
     }