-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to Cef Version 59.0.0 #2114
Changes from 3 commits
35f9cc3
08444fe
2f680f0
ea7d2c8
3d45c10
b5ade51
22b67f0
fb5a0bb
f82d744
62ae9ed
f76f4d9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
<?xml version="1.0" encoding="utf-8"?> | ||
<packages> | ||
<package id="cef.sdk" version="3.3029.1611" targetFramework="Native" /> | ||
<package id="cef.sdk" version="3.3071.1649" targetFramework="native" /> | ||
</packages> |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,7 +20,7 @@ void CefBrowserHostWrapper::DragTargetDragEnter(IDragData^ dragData, MouseEvent | |
|
||
auto dragDataWrapper = static_cast<CefDragDataWrapper^>(dragData); | ||
dragDataWrapper->ResetFileContents(); // Recommended by documentation to reset before calling DragEnter | ||
_browserHost->DragTargetDragEnter(dragDataWrapper, GetCefMouseEvent(mouseEvent), (CefBrowserHost::DragOperationsMask) allowedOperations); | ||
_browserHost->DragTargetDragEnter(*dragDataWrapper, GetCefMouseEvent(mouseEvent), (CefBrowserHost::DragOperationsMask) allowedOperations); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This pointer reference should also be changed. |
||
} | ||
|
||
void CefBrowserHostWrapper::DragTargetDragOver(MouseEvent mouseEvent, DragOperationsMask allowedOperations) | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -25,12 +25,11 @@ void ManagedCefBrowserAdapter::CreateOffscreenBrowser(IntPtr windowHandle, Brows | |
auto hwnd = static_cast<HWND>(windowHandle.ToPointer()); | ||
|
||
CefWindowInfo window; | ||
auto transparent = browserSettings->OffScreenTransparentBackground.GetValueOrDefault(true); | ||
window.SetAsWindowless(hwnd, transparent); | ||
window.SetAsWindowless(hwnd); | ||
CefString addressNative = StringUtils::ToNative(address); | ||
|
||
if (!CefBrowserHost::CreateBrowser(window, _clientAdapter.get(), addressNative, | ||
*browserSettings->_browserSettings, requestContext)) | ||
*browserSettings->_browserSettings, *requestContext)) | ||
{ | ||
throw gcnew InvalidOperationException("Failed to create offscreen browser. Call Cef.Initialize() first."); | ||
} | ||
|
@@ -78,7 +77,7 @@ void ManagedCefBrowserAdapter::CreateBrowser(BrowserSettings^ browserSettings, R | |
CefString addressNative = StringUtils::ToNative(address); | ||
|
||
CefBrowserHost::CreateBrowser(window, _clientAdapter.get(), addressNative, | ||
*browserSettings->_browserSettings, requestContext); | ||
*browserSettings->_browserSettings, *requestContext); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Was there an actual compiler error or were you just going by the As it's a There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Will do I will need to read up on the difference. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
} | ||
|
||
void ManagedCefBrowserAdapter::Resize(int width, int height) | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
<?xml version="1.0" encoding="utf-8"?> | ||
<packages> | ||
<package id="cef.sdk" version="3.3029.1611" targetFramework="Native" /> | ||
<package id="cef.sdk" version="3.3071.1649" targetFramework="native" /> | ||
</packages> |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
<?xml version="1.0" encoding="utf-8"?> | ||
<packages> | ||
<package id="cef.redist.x64" version="3.3029.1611" targetFramework="net452" /> | ||
<package id="cef.redist.x86" version="3.3029.1611" targetFramework="net452" /> | ||
<package id="cef.redist.x64" version="3.3071.1649" targetFramework="net452" /> | ||
<package id="cef.redist.x86" version="3.3071.1649" targetFramework="net452" /> | ||
</packages> |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
<?xml version="1.0" encoding="utf-8"?> | ||
<packages> | ||
<package id="cef.redist.x64" version="3.3029.1611" targetFramework="net452" /> | ||
<package id="cef.redist.x86" version="3.3029.1611" targetFramework="net452" /> | ||
<package id="cef.redist.x64" version="3.3071.1649" targetFramework="net452" /> | ||
<package id="cef.redist.x86" version="3.3071.1649" targetFramework="net452" /> | ||
</packages> |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,7 @@ | ||
<?xml version="1.0" encoding="utf-8"?> | ||
<packages> | ||
<package id="cef.redist.x64" version="3.3029.1611" targetFramework="net452" /> | ||
<package id="cef.redist.x86" version="3.3029.1611" targetFramework="net452" /> | ||
<package id="cef.redist.x64" version="3.3071.1649" targetFramework="net452" /> | ||
<package id="cef.redist.x86" version="3.3071.1649" targetFramework="net452" /> | ||
<package id="CommonServiceLocator" version="1.3" targetFramework="net45" /> | ||
<package id="MvvmLightLibs" version="5.1.1.0" targetFramework="net45" /> | ||
</packages> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The
EnsureNuGetPackageBuildImports
targets need to be removed from this file and the others.See the following for background
http://blog.davidebbo.com/2014/01/the-right-way-to-restore-nuget-packages.html
https://www.xavierdecoster.com/post/2014/03/06/migrate-away-from-msbuild-based-nuget-package-restore.html