WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
135296
[Win][ANGLE] Enable D3D11.
https://bugs.webkit.org/show_bug.cgi?id=135296
Summary
[Win][ANGLE] Enable D3D11.
peavo
Reported
2014-07-25 11:07:54 PDT
Direct3D 11 is not enabled on Windows.
Attachments
Patch
(12.32 KB, patch)
2014-07-25 11:20 PDT
,
peavo
no flags
Details
Formatted Diff
Diff
Patch
(13.17 KB, patch)
2014-07-25 12:27 PDT
,
peavo
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
peavo
Comment 1
2014-07-25 11:20:19 PDT
Created
attachment 235527
[details]
Patch
WebKit Commit Bot
Comment 2
2014-07-25 11:21:16 PDT
Note that there are important steps to take when updating ANGLE. See
http://trac.webkit.org/wiki/UpdatingANGLE
Alex Christensen
Comment 3
2014-07-25 11:25:32 PDT
Comment on
attachment 235527
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=235527&action=review
Does this work with the June 2010 DirectX SDK? I think I tried this and didn't enable it because it couldn't find dxgi1_2.h
> Source/ThirdParty/ANGLE/src/libGLESv2/precompiled.h:51 > +#if !ANGLE_SKIP_DXGI_1_2_CHECK > #include <dxgi1_2.h> > +#endif
Does it really compile without this? Wow. Just remove this line, then, and reflect the changes from ANGLE trunk in changes.diff and don't define ANGLE_SKIP_DXGI_1_2_CHECK.
peavo
Comment 4
2014-07-25 11:58:03 PDT
(In reply to
comment #3
)
> (From update of
attachment 235527
[details]
) > View in context:
https://bugs.webkit.org/attachment.cgi?id=235527&action=review
> > Does this work with the June 2010 DirectX SDK? I think I tried this and didn't enable it because it couldn't find dxgi1_2.h > > > Source/ThirdParty/ANGLE/src/libGLESv2/precompiled.h:51 > > +#if !ANGLE_SKIP_DXGI_1_2_CHECK > > #include <dxgi1_2.h> > > +#endif > > Does it really compile without this? Wow. Just remove this line, then, and reflect the changes from ANGLE trunk in changes.diff and don't define ANGLE_SKIP_DXGI_1_2_CHECK.
I think we need to set ANGLE_SKIP_DXGI_1_2_CHECK = 1, since the define is used as guard in Renderer11.cpp (dxgi check). It seems changes.diff is generated with git. I'm using svn, how do I create the diff?
Alex Christensen
Comment 5
2014-07-25 12:06:09 PDT
Install git, do the command in ANGLE.plist, make your changes, call "git diff", and put that into changes.diff. diff --git a/src/libGLESv2/precompiled.h b/src/libGLESv2/precompiled.h index ceef507..a1da38c 100644 --- a/src/libGLESv2/precompiled.h +++ b/src/libGLESv2/precompiled.h @@ -46,6 +46,8 @@ #include <D3D10_1.h> #include <D3D11.h> #include <dxgi.h> +#if !ANGLE_SKIP_DXGI_1_2_CHECK #include <dxgi1_2.h> +#endif #include <D3Dcompiler.h> #endif // ANGLE_ENABLE_D3D11
peavo
Comment 6
2014-07-25 12:27:55 PDT
Created
attachment 235537
[details]
Patch
peavo
Comment 7
2014-07-25 12:28:23 PDT
(In reply to
comment #5
)
> Install git, do the command in ANGLE.plist, make your changes, call "git diff", and put that into changes.diff. >
Thanks :) Updated patch.
WebKit Commit Bot
Comment 8
2014-07-25 13:07:58 PDT
Comment on
attachment 235537
[details]
Patch Clearing flags on attachment: 235537 Committed
r171610
: <
http://trac.webkit.org/changeset/171610
>
WebKit Commit Bot
Comment 9
2014-07-25 13:08:01 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug