Closed
Bug 1297924
Opened 8 years ago
Closed 8 years ago
Update to ANGLE/2845
Categories
(Core :: Graphics: CanvasWebGL, defect)
Core
Graphics: CanvasWebGL
Tracking
()
RESOLVED
FIXED
mozilla51
People
(Reporter: jrmuizel, Assigned: mtseng)
References
(Blocks 1 open bug)
Details
Attachments
(6 files, 3 obsolete files)
(deleted),
text/x-review-board-request
|
jgilbert
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
jgilbert
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
jgilbert
:
review+
|
Details |
(deleted),
patch
|
ritu
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
ritu
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
ritu
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 2•8 years ago
|
||
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 5•8 years ago
|
||
The ANGLE update PR is here:
https://github.com/mozilla/angle/pull/9
Reporter | ||
Comment 6•8 years ago
|
||
mozreview-review |
Comment on attachment 8786200 [details]
Bug 1297924 - Update ANGLE to chromium/2838.
https://reviewboard.mozilla.org/r/75170/#review73040
Attachment #8786200 -
Flags: review?(jmuizelaar) → review+
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 10•8 years ago
|
||
Latest try with more patches.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=cf3972e03b89
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 14•8 years ago
|
||
try again with latest patch
https://treeherder.mozilla.org/#/jobs?repo=try&revision=a2e49ad4bb17
Assignee | ||
Comment 15•8 years ago
|
||
I suggest update to 2845 which include a patch [1] which fix a problem when we using webgl2 on angle.
[1]: https://github.com/google/angle/commit/4e58af614176072943bdb302b2395dd85a398514
What do you think, jeff?
Flags: needinfo?(jmuizelaar)
Assignee | ||
Comment 16•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e512a3305f95
Latest try with chromium/2845
Reporter | ||
Comment 17•8 years ago
|
||
(In reply to Morris Tseng [:mtseng] [:Morris] from comment #15)
> I suggest update to 2845 which include a patch [1] which fix a problem when
> we using webgl2 on angle.
>
> [1]:
> https://github.com/google/angle/commit/
> 4e58af614176072943bdb302b2395dd85a398514
>
> What do you think, jeff?
I think that's fine. I tried looking to see how Google handles fixing ANGLE problems on the dev and beta channels and couldn't find any evidence of it happening. Given, that it probably makes sense to track closer to Chrome 55 (to be released Dec 6).
Flags: needinfo?(jmuizelaar)
Assignee | ||
Updated•8 years ago
|
Summary: Update to ANGLE/2838 → Update to ANGLE/2845
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Updated•8 years ago
|
Attachment #8786200 -
Attachment is obsolete: true
Assignee | ||
Updated•8 years ago
|
Attachment #8786201 -
Attachment is obsolete: true
Attachment #8786201 -
Flags: review?(jgilbert)
Assignee | ||
Updated•8 years ago
|
Attachment #8786208 -
Attachment is obsolete: true
Attachment #8786208 -
Flags: review?(jgilbert)
Comment 21•8 years ago
|
||
mozreview-review |
Comment on attachment 8788034 [details]
Bug 1297924 - Update shader validator due to interface change.
https://reviewboard.mozilla.org/r/76574/#review75272
Attachment #8788034 -
Flags: review?(jgilbert) → review+
Comment 22•8 years ago
|
||
mozreview-review |
Comment on attachment 8788035 [details]
Bug 1297924 - Remove passed tests from mochitest-errata.
https://reviewboard.mozilla.org/r/76576/#review75274
Attachment #8788035 -
Flags: review?(jgilbert) → review+
Comment 23•8 years ago
|
||
mozreview-review |
Comment on attachment 8788033 [details]
Bug 1297924 - Update ANGLE to chromium/2845.
https://reviewboard.mozilla.org/r/76572/#review75276
Attachment #8788033 -
Flags: review+
Updated•8 years ago
|
Attachment #8788033 -
Flags: review?(jmuizelaar)
Assignee | ||
Comment 24•8 years ago
|
||
Comment 25•8 years ago
|
||
Pushed by mtseng@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/64a77898c426
Update ANGLE to chromium/2845. r=jgilbert
https://hg.mozilla.org/integration/mozilla-inbound/rev/1c9f6480b0e4
Update shader validator due to interface change. r=jgilbert
https://hg.mozilla.org/integration/mozilla-inbound/rev/25f7b5f0dee3
Remove passed tests from mochitest-errata. r=jgilbert
Assignee | ||
Comment 26•8 years ago
|
||
Tracked mozilla angle repo also updated.
https://github.com/mozilla/angle/commits/mozilla
tags: https://github.com/mozilla/angle/commits/mozilla-bug-1297924
Comment 27•8 years ago
|
||
Sorry had to back out for WebGL M(gl3) failure, e.g., https://treeherder.mozilla.org/logviewer.html#?job_id=35428001&repo=mozilla-inbound#L3432
Flags: needinfo?(mtseng)
Comment 28•8 years ago
|
||
Backout by ihsiao@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9d889e3bc5ba
Backed out changeset 25f7b5f0dee3 for webgl test_conformance__textures__misc__copy-tex-image-and-sub-image-2d.html failure
https://hg.mozilla.org/integration/mozilla-inbound/rev/a495d31313af
Backed out changeset 1c9f6480b0e4
https://hg.mozilla.org/integration/mozilla-inbound/rev/c5ed16e1dbc6
Backed out changeset 64a77898c426
Assignee | ||
Comment 29•8 years ago
|
||
Ahha, our tracked angle repo forgot to add this commit in bug 1290634. Add it and it passed test in my local. Will push a new try later.
Flags: needinfo?(mtseng)
Assignee | ||
Comment 30•8 years ago
|
||
Assignee | ||
Comment 31•8 years ago
|
||
Tracked angle repo also been updated.
Assignee | ||
Comment 32•8 years ago
|
||
Try looks good. Windows 7 VM opt fail is known intermittent.
Comment 33•8 years ago
|
||
Pushed by mtseng@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/53735145cfbe
Update ANGLE to chromium/2845. r=jgilbert
https://hg.mozilla.org/integration/mozilla-inbound/rev/d065b4a4e420
Update shader validator due to interface change. r=jgilbert
https://hg.mozilla.org/integration/mozilla-inbound/rev/1c1d78ab773c
Remove passed tests from mochitest-errata. r=jgilbert
Comment 34•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/53735145cfbe
https://hg.mozilla.org/mozilla-central/rev/d065b4a4e420
https://hg.mozilla.org/mozilla-central/rev/1c1d78ab773c
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox51:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
Assignee | ||
Comment 35•8 years ago
|
||
MozReview-Commit-ID: 90sB4crXQDl
Assignee | ||
Comment 36•8 years ago
|
||
MozReview-Commit-ID: 8VSrUepZsmz
Assignee | ||
Comment 37•8 years ago
|
||
MozReview-Commit-ID: 6OwLEPtdxzA
Assignee | ||
Comment 38•8 years ago
|
||
Comment on attachment 8791074 [details] [diff] [review]
Update ANGLE to chromium/2845. r=jgilbert
This is dependent patch for bug 1297965, please see bug 1297965 comment 25.
Attachment #8791074 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 39•8 years ago
|
||
Comment on attachment 8791075 [details] [diff] [review]
Update shader validator due to interface change. r=jgilbert
This is dependent patch for bug 1297965, please see bug 1297965 comment 25.
Attachment #8791075 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 40•8 years ago
|
||
Comment on attachment 8791076 [details] [diff] [review]
Remove passed tests from mochitest-errata. r=jgilbert
This is dependent patch for bug 1297965, please see bug 1297965 comment 25.
Attachment #8791076 -
Flags: approval-mozilla-aurora?
Comment on attachment 8791074 [details] [diff] [review]
Update ANGLE to chromium/2845. r=jgilbert
WebGL2 support, Aurora50+
Attachment #8791074 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
status-firefox50:
--- → affected
Attachment #8791075 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Attachment #8791076 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 42•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/9ca6bbc6fdb3
https://hg.mozilla.org/releases/mozilla-aurora/rev/8cff6e89a868
https://hg.mozilla.org/releases/mozilla-aurora/rev/b63f2bd9ccdf
Flags: in-testsuite+
Comment 43•8 years ago
|
||
Backed out for Windows 7 test_2_conformance2__reading__read-pixels-pack-parameters.html asserts.
https://treeherder.mozilla.org/logviewer.html#?job_id=3562258&repo=mozilla-aurora
Assertion failed: mMemoryBuffer.size() > 0, file c:/builds/moz2_slave/m-aurora-w32-d-000000000000000/build/src/gfx/angle/src/libANGLE/renderer/d3d/d3d11/Buffer11.cpp, line 1375
https://hg.mozilla.org/releases/mozilla-aurora/rev/2c332306c030
Flags: needinfo?(mtseng)
Comment 44•8 years ago
|
||
bugherder uplift |
Assignee | ||
Updated•8 years ago
|
Flags: needinfo?(mtseng)
You need to log in
before you can comment on or make changes to this bug.
Description
•