Closed
Bug 1143924
Opened 9 years ago
Closed 9 years ago
Remove added code from generated wrapper files
Categories
(Core :: Graphics: CanvasWebGL, defect)
Tracking
()
RESOLVED
FIXED
mozilla39
Tracking | Status | |
---|---|---|
firefox39 | --- | fixed |
People
(Reporter: jgilbert, Assigned: jgilbert)
References
Details
Attachments
(1 file, 2 obsolete files)
207.95 KB,
patch
|
u480271
:
review+
|
Details | Diff | Splinter Review |
Some extra code has snuck in, and we should move it back out.
Attachment #8578328 -
Flags: review?(dglastonbury)
Comment on attachment 8578328 [details] [diff] [review] 0001-Remove-code-additions-from-generated-wrappers.patch Review of attachment 8578328 [details] [diff] [review]: ----------------------------------------------------------------- r+ with green try run.
Attachment #8578328 -
Flags: review?(dglastonbury) → review+
Assignee | ||
Comment 2•9 years ago
|
||
Let's try to warn people off of editing our generated files.
Attachment #8578789 -
Flags: review?(dglastonbury)
Assignee | ||
Comment 3•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=2423b37f6ecf
Comment on attachment 8578789 [details] [diff] [review] 0002-Add-GENERATED-FILE-comment-to-_wrapper-files.patch Review of attachment 8578789 [details] [diff] [review]: ----------------------------------------------------------------- Maybe split the generated files from the changed files next time.
Attachment #8578789 -
Flags: review?(dglastonbury) → review+
Assignee | ||
Comment 5•9 years ago
|
||
Ok, so there were some false starts in there. Here's what passes on Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=faf7907d1fc7
Attachment #8578328 -
Attachment is obsolete: true
Attachment #8578789 -
Attachment is obsolete: true
Attachment #8580477 -
Flags: review?(dglastonbury)
Attachment #8580477 -
Flags: review?(dglastonbury) → review+
(In reply to Jeff Gilbert [:jgilbert] from comment #5) > Created attachment 8580477 [details] [diff] [review] > fix-generated.diff > > Ok, so there were some false starts in there. Here's what passes on Try: > https://treeherder.mozilla.org/#/jobs?repo=try&revision=faf7907d1fc7 Might be worth throwing in 10.8 and 10.10 for testing too.
Assignee | ||
Comment 7•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/7d9bc7d19984
Comment 8•9 years ago
|
||
sorry had to back this out for test failures like https://treeherder.mozilla.org/logviewer.html#?job_id=7926689&repo=mozilla-inbound
Flags: needinfo?(jgilbert)
Assignee | ||
Comment 9•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ebd993b59b8f https://hg.mozilla.org/integration/mozilla-inbound/rev/cd28897da642
Flags: needinfo?(jgilbert)
Comment 10•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/cd28897da642
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox39:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
You need to log in
before you can comment on or make changes to this bug.
Description
•