pageshow event does not follow spec

Do not change pageshow event behavior though it does not follow spec

As per spec step of history traversal algorithm (section 7.6.10): subsection 4.6 of 
http://www.whatwg.org/specs/web-apps/current-work/multipage/history.html#traverse-the-history 
browser should queue a task to fire pageshow event. However chrome and other desktop browsers 
firefox30 and IE11) dispatches the event sync right after onload is triggered. 
Specification change requested for the same so just update the same in comments comments. 

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

[maheshkk, 2014-05-29 21:12:38]

PTAL

[Inactive, 2014-05-29 21:19:39]

Please add an API OWNER as reviewer.

https://codereview.chromium.org/294663007/diff/40001/Source/core/frame/DOMWin...
File Source/core/frame/DOMWindow.cpp (right):

https://codereview.chromium.org/294663007/diff/40001/Source/core/frame/DOMWin...
Source/core/frame/DOMWindow.cpp:456: // As per spec pageshow must be triggered
asyncronously.
asynchronously

[maheshkk, 2014-05-29 21:27:20]

PTAL 
More details and screenshot at
https://code.google.com/p/chromium/issues/detail?id=378931

[tkent, 2014-05-30 01:05:49]

https://codereview.chromium.org/294663007/diff/40001/Source/core/frame/DOMWin...
File Source/core/frame/DOMWindow.cpp (left):

https://codereview.chromium.org/294663007/diff/40001/Source/core/frame/DOMWin...
Source/core/frame/DOMWindow.cpp:456: // FIXME:
https://bugs.webkit.org/show_bug.cgi?id=36334 Pageshow event needs to fire
asynchronously.
IMO we should not remove the FIXME. We should add a sentence about IE and
Firefox behavior.
Also, we should talk to the standard body.

[maheshkk, 2014-05-30 21:36:13]

On 2014/05/30 01:05:49, tkent wrote:
>
https://codereview.chromium.org/294663007/diff/40001/Source/core/frame/DOMWin...
> File Source/core/frame/DOMWindow.cpp (left):
> 
>
https://codereview.chromium.org/294663007/diff/40001/Source/core/frame/DOMWin...
> Source/core/frame/DOMWindow.cpp:456: // FIXME:
> https://bugs.webkit.org/show_bug.cgi?id=36334 Pageshow event needs to fire
> asynchronously.
> IMO we should not remove the FIXME. We should add a sentence about IE and
> Firefox behavior.
> Also, we should talk to the standard body.

Thanks for the review. I have raised a bug with specification here
https://www.w3.org/Bugs/Public/show_bug.cgi?id=25929

PTAL

[tkent, 2014-06-01 23:20:09]

lgtm.
Please update the CL description before landing.

[maheshkk, 2014-06-02 16:49:35]

The CQ bit was checked by mahesh.kk@samsung.com

[commit-bot: I haz the power, 2014-06-02 16:49:55]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/mahesh.kk@samsung.com/294663007/80001

[commit-bot: I haz the power, 2014-06-02 18:05:47]

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

[commit-bot: I haz the power, 2014-06-02 18:52:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-02 18:52:39]

Try jobs failed on following builders:
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/9897)

[maheshkk, 2014-06-02 18:55:12]

The CQ bit was checked by mahesh.kk@samsung.com

[commit-bot: I haz the power, 2014-06-02 18:55:47]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/mahesh.kk@samsung.com/294663007/80001

[commit-bot: I haz the power, 2014-06-02 21:25:53]

Change committed as 175311