third_party/WebKit/LayoutTests/inspector/workspace-mapping-expected.txt - Issue 2349343002: DevTools: introduce persistence/ module

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: third_party/WebKit/LayoutTests/inspector/workspace-mapping-expected.txt

Issue 2349343002: DevTools: introduce persistence/ module (Closed)

Patch Set: reupload Created 4 years, 2 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

« third_party/WebKit/LayoutTests/http/tests/inspector/persistence/persistence-test.js ('K') | « third_party/WebKit/LayoutTests/inspector/workspace-mapping.html ('k') | third_party/WebKit/Source/devtools/BUILD.gn » ('j') | third_party/WebKit/Source/devtools/front_end/persistence/Persistence.js » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 Tests workspace mappings	1 Tests workspace mappings

2	2

3 url http://www.example.com/index.html is not mapped.	3 url http://www.example.com/index.html is not mapped.

4 url http://localhost/index.html is mapped.	4 url http://localhost/index.html is mapped.

5 url http://localhost/foo/index.html is mapped.	5 url http://localhost/foo/index.html is mapped.

6 url https://localhost/index.html is not mapped.	6 url https://localhost/index.html is not mapped.

7	7

8 url http://www.example.com/index.html is mapped to null	8 url http://www.example.com/index.html is mapped to null

9 url http://localhost/index.html is mapped to file:///var/www/localhost/index .html	9 url http://localhost/index.html is mapped to null
	dgozman 2016/09/23 23:03:21 suspicicous suspicicous lushnikov 2016/09/24 00:30:50 This happens for the same reason as the rest of th Show quoted text On 2016/09/23 23:03:21, dgozman wrote: > suspicicous This happens for the same reason as the rest of the calls in this block. Since we don't query fileMapping any more, the workspace doesn't have these urls. dgozman 2016/09/24 02:34:34 I see. But this means uiSourceCodeForURL(url) retu Show quoted text On 2016/09/24 00:30:50, lushnikov wrote: > On 2016/09/23 23:03:21, dgozman wrote: > > suspicicous > > This happens for the same reason as the rest of the calls in this block. > Since we don't query fileMapping any more, the workspace doesn't have these > urls. I see. But this means uiSourceCodeForURL(url) return null which is strange. Could you please double-check and maybe change the test?
10 url http://localhost/foo/index.html is mapped to null	10 url http://localhost/foo/index.html is mapped to null

11 url https://localhost/index.html is mapped to null	11 url https://localhost/index.html is mapped to null

12	12

13 path file:///home/example.com / foo/index.html is mapped to null	13 path file:///home/example.com / foo/index.html is mapped to null

14 path file:///home/example.com / index.html is mapped to null	14 path file:///home/example.com / index.html is mapped to null

15 path file:///var/www / localhost/index.html is mapped to http://localhost/in dex.html	15 path file:///var/www / localhost/index.html is mapped to http://localhost/in dex.html

16 path file:///var/www / localhost/foo/index.html is mapped to http://localhos t/foo/index.html	16 path file:///var/www / localhost/foo/index.html is mapped to http://localhos t/foo/index.html

17 path file:///home/foo / index.html is mapped to null	17 path file:///home/foo / index.html is mapped to null

18	18

OLD	NEW