Reset DocumentLoader::m_request's method to GET for pushState and replaceState

Reset DocumentLoader::m_request's method to GET for pushState and replaceState

We currently correctly send a GET in the POST->replaceState->reload case, but
DocumentLoader::m_request still shows a method of POST, and as a result we
incorrectly report it as a POST to the browser process. This contributes to
unnecessarily showing a form resubmission warning in that case.

BUG=138324
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175011

[Nate Chapin, 2014-05-28 21:26:15]

https://codereview.chromium.org/302543008/diff/20001/Source/core/loader/Frame...
File Source/core/loader/FrameLoader.cpp (left):

https://codereview.chromium.org/302543008/diff/20001/Source/core/loader/Frame...
Source/core/loader/FrameLoader.cpp:288:
m_currentItem->setFormInfoFromRequest(isPushOrReplaceState ? ResourceRequest() :
m_documentLoader->request());
This special case is no longer necessary, as setFormInfoFromRequest will
naturally do the right thing for pushState/replaceState now that
DocumentLoader::request() is a GET.

[dglazkov, 2014-05-28 21:38:06]

lgtm

pretty.

[Nate Chapin, 2014-05-28 21:39:34]

The CQ bit was checked by japhet@chromium.org

[commit-bot: I haz the power, 2014-05-28 21:39:47]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/japhet@chromium.org/302543008/20001

[commit-bot: I haz the power, 2014-05-28 22:45:16]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/9541)

[commit-bot: I haz the power, 2014-05-28 23:35:44]

Change committed as 175011

[Nate Chapin, 2014-05-28 23:49:19]

On 2014/05/28 23:35:44, I haz the power (commit-bot) wrote:
> Change committed as 175011

Oops. That was supposed to be
https://code.google.com/p/chromium/issues/detail?id=239907 in the commit
message.