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
Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr)
|
Please choose your nickname with
Settings
|
Help
|
Chromium Project
|
Gerrit Changes
|
Sign out
(128)
Issues
Search
My Issues
|
Starred
Open
|
Closed
|
All
Side by Side Diff: test/inspector/debugger/restore-breakpoint-expected.txt
Issue
2671193002
:
[inspector] restore provisional breakpoints smarter (Closed)
Patch Set: addressed comments
Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Draft comments are only viewable by you.
Context:
3 lines
10 lines
25 lines
50 lines
75 lines
100 lines
Whole file
Column Width:
Tab Spaces:
Jump to:
src/api.cc
src/debug/debug-interface.h
src/inspector/v8-debugger-agent-impl.h
src/inspector/v8-debugger-agent-impl.cc
src/inspector/v8-debugger-script.h
src/inspector/v8-debugger-script.cc
test/inspector/debugger/restore-breakpoint.js
test/inspector/debugger/restore-breakpoint-expected.txt
View unified diff
|
Download patch
« src/inspector/v8-debugger-agent-impl.cc
('K') |
« test/inspector/debugger/restore-breakpoint.js
('k') |
no next file »
|
no next file with comments »
Toggle Intra-line Diffs
('i') |
Expand Comments
('e') |
Collapse Comments
('c') |
Show Comments
Hide Comments
('s')
OLD
NEW
(Empty)
1
Checks that debugger agent uses source content to restore breakpoints.
2
3
Running test: testSameSource
4
function foo() {
5
#boo();
6
}
7
function foo() {
8
#boo();
9
}
10
11
Running test: testOneLineOffset
12
function foo() {
13
#boo();
14
}
15
function foo() {
16
#boo();
17
boo();
18
}
19
20
Running test: testTwoSimilarLinesCloseToOriginalLocation1
21
function foo() {
22
23
24
#boo();
25
}
26
function foo() {
27
#boo();
28
29
newCode();
30
boo();
31
32
33
34
boo();
35
}
36
37
Running test: testTwoSimilarLinesCloseToOriginalLocation2
38
function foo() {
39
40
41
#boo();
42
}
43
function foo() {
44
boo();
45
newLongCode();
46
newCode();
47
#boo();
48
49
50
51
boo();
52
}
53
54
Running test: testHintIgnoreWhiteSpaces
55
function foo() {
56
57
58
59
#boo();
60
}
61
function foo() {
62
foo();
63
64
65
#boo();
66
}
67
68
Running test: testCheckOnlyLimitedOffsets
69
function foo() {
70
#boo();
71
}
72
function foo() {
73
#newCode();
74
;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;...
75
boo();
76
}
OLD
NEW
« src/inspector/v8-debugger-agent-impl.cc
('K') |
« test/inspector/debugger/restore-breakpoint.js
('k') |
no next file »
|
no next file with comments »
Issue 2671193002: [inspector] restore provisional breakpoints smarter (Closed)
Created 3 years, 10 months ago by kozy
Modified 3 years, 9 months ago
Reviewers: pfeldman, alph, Yang
Base URL:
Comments: 46
This is Rietveld
408576698