4 years, 11 months ago
(2015-06-02 04:56:22 UTC)
#3
jsbell, would you review this please?
jsbell
lgtm https://codereview.chromium.org/1165753005/diff/20001/LayoutTests/imported/web-platform-tests/html/browsers/history/the-history-interface/007-expected.txt File LayoutTests/imported/web-platform-tests/html/browsers/history/the-history-interface/007-expected.txt (left): https://codereview.chromium.org/1165753005/diff/20001/LayoutTests/imported/web-platform-tests/html/browsers/history/the-history-interface/007-expected.txt#oldcode1 LayoutTests/imported/web-platform-tests/html/browsers/history/the-history-interface/007-expected.txt:1: It looks like the browser stopped loading the ...
4 years, 11 months ago
(2015-06-02 17:34:21 UTC)
#4
https://codereview.chromium.org/1165753005/diff/20001/LayoutTests/imported/web-platform-tests/html/browsers/history/the-history-interface/007-expected.txt File LayoutTests/imported/web-platform-tests/html/browsers/history/the-history-interface/007-expected.txt (left): https://codereview.chromium.org/1165753005/diff/20001/LayoutTests/imported/web-platform-tests/html/browsers/history/the-history-interface/007-expected.txt#oldcode1 LayoutTests/imported/web-platform-tests/html/browsers/history/the-history-interface/007-expected.txt:1: It looks like the browser stopped loading the page ...
4 years, 11 months ago
(2015-06-02 20:41:15 UTC)
#5
https://codereview.chromium.org/1165753005/diff/20001/LayoutTests/imported/we...
File
LayoutTests/imported/web-platform-tests/html/browsers/history/the-history-interface/007-expected.txt
(left):
https://codereview.chromium.org/1165753005/diff/20001/LayoutTests/imported/we...
LayoutTests/imported/web-platform-tests/html/browsers/history/the-history-interface/007-expected.txt:1:
It looks like the browser stopped loading the page when encountering a .go(-1)
command pointing to a pushed state. This will break the tests.
On 2015/06/02 17:34:21, jsbell wrote:
> Is this bogus, anything we need to investigate, or is it covered by existing
> bugs or test cases below?
I think the FAIL lines are due to this behavior. I'll revisit all of FAILs in
-expected.txt later.
tkent
The CQ bit was checked by tkent@chromium.org
4 years, 11 months ago
(2015-06-02 20:41:19 UTC)
#6
Issue 1165753005: testharnessreport: Remove div[id=log] from resultant HTML.
(Closed)
Created 4 years, 11 months ago by tkent
Modified 4 years, 11 months ago
Reviewers: jsbell
Base URL: svn://svn.chromium.org/blink/trunk
Comments: 2