This repository was archived by the owner on Sep 2, 2021. It is now read-only.
Merge master into release for 1.8 release #592
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The gyp/test folder was excluded from the commit.
This is the same fix of Glenn Ruehle of 18/06/2013 on top of the update of gyp (SHA-1: a276d55)
Updating verison number to 1.8 on master
Moved client_switches files under a new common folder
VS2015: add link libraries
Removed string_util since is unused
Replaced util macros in favour of cef_helpers|cef_logging ones
VS2015: fix some warnings
Updated gyp
Fix for liner error on VS 2013
Update common.gypi
Change path of xdg-open to match opn
Update compatibility.manifest for Windows 10
Update .gitignore
Rework a bit appshell gyp files
Some cleanup - Make the license evergreen - Remove trailing whitespaces from license comments - Reorder a bit the includes and add a newline after the license
Remove define OS_* and use config.h instead
Add missing include and fix path libcef.so
Ignore only toplevel folder and add out folder too
Move AppShellExtensionHandler and StContextScope out of client_app
Move fixupKey where it is used and fix headers inclusion
Import cefclient-2623 code and include it on Linux build only
Appshell helpers
Init command_line directly in cefclient_<platform>
Move more things from cefclient_gtk to appshell_helpers
[Win] Fixing compilation error
ficristo
approved these changes
Oct 24, 2016
|
I looked only on Windows and it is good for me. |
|
Merging this PR for 1.8 release. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@nethip @ficristo Please have a look at the commits once.