Chromium Code Reviews
Help | Chromium Project | Sign in
(746)

Issue 1584008: Let LiveEdit accept a full new script source (rather than diff) (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years ago by Peter Rybin
Modified:
2 years, 11 months ago
CC:
v8-dev_googlegroups.com
Visibility:
Public.

Description

Let LiveEdit accept a full new script source (rather than diff)

Committed: http://code.google.com/p/v8/source/detail?r=4338

Patch Set 1 #

Patch Set 2 : clean up #

Total comments: 4

Patch Set 3 : follow codereview #

Unified diffs Side-by-side diffs Delta from patch set Stats (+155 lines, -10 lines) Lint Patch
M src/debug-debugger.js View 1 2 2 chunks +25 lines, -10 lines 0 comments ? errors Download
M src/liveedit-debugger.js View 1 1 chunk +83 lines, -0 lines 0 comments ? errors Download
A test/mjsunit/debug-liveedit-newsource.js View 1 chunk +47 lines, -0 lines 0 comments ? errors Download
Trybot results:
Commit:

Messages

Total messages: 3
Peter Rybin
4 years ago #1
Mikhail Naganov (Cr)
LGTM http://codereview.chromium.org/1584008/diff/2001/3001 File src/debug-debugger.js (right): http://codereview.chromium.org/1584008/diff/2001/3001#newcode2008 src/debug-debugger.js:2008: invocation(); nit: alignment http://codereview.chromium.org/1584008/diff/2001/3002 File src/liveedit-debugger.js (right): http://codereview.chromium.org/1584008/diff/2001/3002#newcode446 ...
4 years ago #2
Peter Rybin
4 years ago #3
http://codereview.chromium.org/1584008/diff/2001/3001
File src/debug-debugger.js (right):

http://codereview.chromium.org/1584008/diff/2001/3001#newcode2008
src/debug-debugger.js:2008: invocation();
On 2010/04/02 09:05:55, Michail Naganov wrote:
> nit: alignment

Done.

http://codereview.chromium.org/1584008/diff/2001/3002
File src/liveedit-debugger.js (right):

http://codereview.chromium.org/1584008/diff/2001/3002#newcode446
src/liveedit-debugger.js:446: new_source.substring(diff.change_pos,
diff.change_pos + diff.new_len),
On 2010/04/02 09:05:55, Michail Naganov wrote:
> nit: you could use 'substr' method, which accepts length as its second
argument.

I thought this is not strictly standard.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6