Closed Bug 657748 Opened 9 years ago Closed 9 years ago

Update ANGLE to r653

Categories

(Core :: Canvas: WebGL, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla6

People

(Reporter: bjacob, Assigned: bjacob)

References

Details

Attachments

(13 files)

111 bytes, patch
Details | Diff | Splinter Review
111 bytes, patch
Details | Diff | Splinter Review
111 bytes, patch
Details | Diff | Splinter Review
111 bytes, patch
Details | Diff | Splinter Review
111 bytes, patch
Details | Diff | Splinter Review
111 bytes, patch
Details | Diff | Splinter Review
111 bytes, patch
Details | Diff | Splinter Review
6.43 KB, patch
Details | Diff | Splinter Review
13.57 KB, patch
Details | Diff | Splinter Review
66.96 KB, patch
Details | Diff | Splinter Review
1.51 KB, patch
Details | Diff | Splinter Review
1.44 KB, patch
Details | Diff | Splinter Review
2.60 KB, patch
Details | Diff | Splinter Review
Contrary to what we believed, newer ANGLE does not require the June 2010 DirectX SDK.
This is just a placeholder for reversing r68002 (update of ANGLE to r611)
Attachment #533050 - Flags: review?(joe)
Just a placeholder for reversing r68001 (update ANGLE to r551)
Attachment #533051 - Flags: review?(joe)
Attachment #533051 - Attachment description: part 1. [PLACEHOLDER] Unapply r68001 (update ANGLE to r551) → part 2. [PLACEHOLDER] Unapply r68001 (update ANGLE to r551)
placeholder for reversing r68000 (ANGLE Makefiles)
Attachment #533052 - Flags: review?(joe)
placeholder for reversing r67999 (renaming debug->compilerdebug)
Attachment #533053 - Flags: review?(joe)
placeholder for unapplying our local ANGLE patches:
  angle-fixes.patch
  angle-nspr-misc.patch
  angle-pbuffers.patch
  angle-shared.patch
Attachment #533055 - Flags: review?(joe)
placeholder for applying the ANGLE diff from r550 to r653.
Attachment #533057 - Flags: review?(joe)
placeholder for reapplying angle-nspr-misc.patch and angle-shared.patch.

The other two patches we had have been merged upstream.
Attachment #533058 - Flags: review?(joe)
Attachment #533058 - Attachment description: part 5. [PLACEHOLDER] Reapply local ANGLE patches (angle-nspr-misc and angle-shared) → part 7. [PLACEHOLDER] Reapply local ANGLE patches (angle-nspr-misc and angle-shared)
In addition to a few new .cpp files, we have to link to D3DCompiler.lib.
Attachment #533065 - Flags: review?(joe)
This will make future updates easier.
Attachment #533066 - Flags: review?(joe)
This is a fix for ANGLE issue, http://code.google.com/p/angleproject/issues/detail?id=158

It works around a MSVS 2005 build problem.
Attachment #533346 - Flags: review?(joe)
At this point, what's blocking us from proceeding with this ANGLE upgrade is that newer ANGLE requires the June 2010 SDK, that's not installed on our builders.
See http://code.google.com/p/angleproject/issues/detail?id=158
and bug 648804.
Depends on: 648804
Duplicate of this bug: 646229
Blocks: 657619
This fixes an orange I had on linux test slaves: this sub-test was crashing the NVIDIA 190.42 linux driver. Not worth fighting against. The reason why that wasn't crashing before is that ANGLE used to block that and now it forwards it to the GL.
Attachment #534741 - Flags: review?(joe)
Tested on your WinXP laptop.
Attachment #534748 - Flags: review?(joe)
Attachment #533050 - Flags: review?(joe)
Attachment #533051 - Flags: review?(joe)
Attachment #533052 - Flags: review?(joe)
Attachment #533053 - Flags: review?(joe)
Attachment #533055 - Flags: review?(joe)
Attachment #533057 - Flags: review?(joe)
Attachment #533058 - Flags: review?(joe)
Attachment #533060 - Flags: review?(joe)
Attachment #533065 - Flags: review?(joe)
Attachment #533066 - Flags: review?(joe)
Attachment #533346 - Flags: review?(joe)
Attachment #534741 - Flags: review?(joe)
Attachment #534748 - Flags: review?(joe)
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Assignee: nobody → bjacob
Target Milestone: --- → mozilla6
Depends on: 674731
You need to log in before you can comment on or make changes to this bug.