Re: Issue 558998 in chromium: Deprecate //@ sourceURL & bug with multiple sourceURL

1,664 views
Skip to first unread message

chro...@googlecode.com

unread,
Feb 3, 2016, 6:14:51 PM2/3/16
to chromi...@chromium.org
Updates:
Summary: Deprecate //@ sourceURL & bug with multiple sourceURL

Comment #17 on issue 558998 by paulir...@chromium.org: Deprecate //@
sourceURL & bug with multiple sourceURL
https://code.google.com/p/chromium/issues/detail?id=558998

sg.

After, we can put in a console warning about //@.
After that, it's likely we wont be able to remove for 6-12 months. Ah well.

--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

chro...@googlecode.com

unread,
Feb 9, 2016, 5:56:16 PM2/9/16
to chromi...@chromium.org
Updates:
Labels: -merge-requested-4.9 Merge-Request-49

Comment #18 on issue 558998 by amin...@google.com: Deprecate //@ sourceURL
bad merge label, applying proper label

chro...@googlecode.com

unread,
Feb 9, 2016, 6:00:18 PM2/9/16
to chromi...@chromium.org
Updates:
Labels: -Merge-Request-49 merge-requested-4.9

Comment #19 on issue 558998 by amin...@google.com: Deprecate //@ sourceURL
Apologies, I see this was a V8 request, perhaps they follow a different
flow - I will follow up with hablich@ and confirm. Reapplying original
label in the meantime.

chro...@googlecode.com

unread,
Feb 17, 2016, 9:38:36 PM2/17/16
to chromi...@chromium.org
Updates:
Cc: habl...@chromium.org

Comment #20 on issue 558998 by kozyat...@chromium.org: Deprecate //@
sourceURL & bug with multiple sourceURL
https://code.google.com/p/chromium/issues/detail?id=558998

@hablich, please take a look. I need merge approval for M49.

chro...@googlecode.com

unread,
Feb 18, 2016, 3:04:39 PM2/18/16
to chromi...@chromium.org
Updates:
Labels: -merge-requested-4.9 merge-approved-4.9
Cr-Blink-JavaScript-Runtime Merge-Request-49

Comment #21 on issue 558998 by habl...@chromium.org: Deprecate //@
sourceURL & bug with multiple sourceURL
https://code.google.com/p/chromium/issues/detail?id=558998

Please make sure to wait for approval for the Chromium parts.

V8 parts approved.

chro...@googlecode.com

unread,
Feb 18, 2016, 7:17:05 PM2/18/16
to chromi...@chromium.org
Updates:
Labels: -Merge-Request-49 Merge-Review-49 Hotlist-Merge-Review

Comment #23 on issue 558998 by tin...@google.com: Deprecate //@ sourceURL &
bug with multiple sourceURL
https://code.google.com/p/chromium/issues/detail?id=558998#c23

[Automated comment] Reverts referenced in bugdroid comments, after merge
request, needs manual review.
Reply all
Reply to author
Forward
0 new messages