[CLOSED] Fix CEF 2171 build on Linux

[core-ai-bot]

Issue by jasonsanjose
Thursday Jan 22, 2015 at 11:19 GMT
Originally opened as adobe#499

---

Replaces adobe#489.

Manually merge changes to cefclient sample app into our linux implementation.

• GYP file changes https://code.google.com/p/chromiumembedded/source/browse/branches/2171/cef3/cef.gyp
• Client app changes https://code.google.com/p/chromiumembedded/source/browse/branches/2171/cef3/tests/cefclient/

---

jasonsanjose included the following code: https://github.com/adobe/brackets-shell/pull/499/commits

[core-ai-bot]

Comment by ingorichter
Thursday Jan 29, 2015 at 06:59 GMT

---

I build this branch and it looks partly good, although better than before.

• 👍 Terminating Brackets via the File->Quit works fine
• 👎 closing the Brackets window results in a crash ([0128/225630:FATAL:cef_ref_counted.h(339)] Assert failed: ptr_ != __null.)

I have tested this on Ubuntu 14.10 32 Bit

[core-ai-bot]

Comment by nethip
Thursday Jan 29, 2015 at 07:38 GMT

---

@ingorichter yeah we ran into this too, while we were doing some unit testing. I am on it. I kind of figured out the problem. I will push more changes to this branch with my changes which should solve the problem.

[core-ai-bot]

Comment by nethip
Thursday Jan 29, 2015 at 17:35 GMT

---

@ingorichter @jasonsanjose I fixed this problem partially but this bug has lot of side effects(Especially when dealing with multiple Brackets windows). This does not seem to be an easy fix. Like @jasonsanjose said, the latest version of cefclient is a lot different than our appshell. I think the right fix is to pick the latest linux counter parts of cefclient and then integrate our appshell changes into it. This definently is a big task. However I will see if we can fix this so that the side effects are minimal. In whatever case, I will update this thread with my findings and hopefully some solution.

[core-ai-bot]

Comment by nethip
Friday Jan 30, 2015 at 12:37 GMT

---

@ingorichter @jasonsanjose I have managed to fix this problem. My latest commit to this branch has the solution for crash on quit. Could you guys pull down the latest changes and do some testing around this.

[core-ai-bot]

Comment by jasonsanjose
Friday Jan 30, 2015 at 18:51 GMT

---

Closing via the window X button works great now, but I see a hang when quitting via File > Quit now where it was previously working fine. Almost there though!

[core-ai-bot]

Comment by nethip
Friday Jan 30, 2015 at 19:19 GMT

---

@jasonsanjose That's strange. I am not able to reproduce this crash. In fact the new commit was to fix both the crashes. I don't know if I missed something in my commit.

If you are still able to crash Brackets with my new commit, I am going to have to power up my Linux system again to double check this. Bad thing I will have to wait until Monday as the Linux system is in the office. 😞

[core-ai-bot]

Comment by nethip
Tuesday Feb 03, 2015 at 17:59 GMT

---

@jasonsanjose @ingorichter at last I managed to repro the issue. The crash is evident only on VM. I am using VMWare Fusion. Weird thing that I observed is that, this crash is reproducible when the VM image's no of processors setting is set to 1. If I change it to 2, I am not able to repro the crash. Also I found another defect while doing code review, the fix for which I have already pushed in, though this does not fix the bug with crash/hang, when VM image's no of processors setting is set to 1.

Since the the hang is happening with VM, should we go ahead and merge this branch into master?

[core-ai-bot]

Comment by jasonsanjose
Tuesday Feb 03, 2015 at 21:20 GMT

---

Ok with me.

Has anyone else tested these changes to make sure mac and windows still build without any new issues? It's worth sanity checking since we did change some cross-platform code here as well as the gyp files (though that's mostly isolated to the linux setup).

[core-ai-bot]

Comment by jasonsanjose
Friday Feb 06, 2015 at 00:45 GMT

---

I tried building this branch on mac and it fails. I must have broken something in the gyp build. I'll take a look.

$ grunt build
Running "build" task

Running "build-mac" task
>> ** BUILD FAILED **
>> 
>> 
>> The following build commands failed:
>>  CompileC xcodebuild/appshell.build/Release/appshell_helper_app.build/Objects-normal/i386/appshell_extensions.o appshell/appshell_extensions.cpp normal i386 c++ com.apple.compilers.llvm.clang.1_0.compiler
>> (1 failure)
Warning: Task "build-mac" failed. Use --force to continue.

Aborted due to warnings.

[core-ai-bot]

Comment by nethip
Friday Feb 06, 2015 at 06:23 GMT

---

@jasonsanjose I figured this out too. This was because we have added an AfterClose() OVERRIDE in client_handler.h and this is only overridden for the Linux platform. We could take the OVERRIDE out and subsitute it with empty method/call the base class version, so the clients interested in can override this. Even better, take this off, as this method is not getting called on Linux platform(in which case we will have to remove the definition in client_handler_gtk.cpp).

[core-ai-bot]

Comment by jasonsanjose
Friday Feb 06, 2015 at 17:39 GMT

---

Good idea. I removed AfterClose. Confirmed both mac and linux builds complete and tested app quit on mac and linux.

[core-ai-bot]

Comment by lrebrown
Tuesday Feb 10, 2015 at 19:56 GMT

---

I've just tried a new 1.2 preview (v1.2.0-15668 master 027de3046) in a Debian sid VM. Initially all was well, except it not remembering window size/position upon reopening. However now it's crashing on close, though only with the 'x' button it seems (which worked initially). File > quit continues to work fine.

In system manager I can see four brackets processes:

• 'brackets'
• 'brackets-node'
• 'brackets --type=zygote --no-sandbox --lang=en-US --log-severity=disable'
• 'brackets --type=renderer --no-sandbox --enable-deferred-image-decoding --lang=en-US --lang=en-US --log-severity=disable --enable-delegated-renderer --enable-impl-side-painting --num-raster-threads=1 --disable-accelerated-video-decode --channel=31900.1.2105914780'

When the problem occurs, the last of these disappears, the other three remain.

The repeated lang param is not a typo btw ;)

I'm running this VM in virtualbox, and I'm using gnome. The VM has 1 CPU core.

[core-ai-bot]

Comment by Romane-T
Wednesday Feb 11, 2015 at 00:15 GMT

---

Good morning

See also thread adobe/brackets#10255 re: libcrypt version breaks Brackets

As per posting in that thread from Jason, installed and tested build at https://github.com/adobe/brackets/releases/download/linux-cef-2171/Brackets.Release.1.2.64-bit.deb. For the very first time since have been using Brackets, it closes cleanly on the first click on the 'x'. Closing via 'File->Quit' still closing cleanly as before.

Debian Jessie with KDE as desktop environment.

With greetings

Romane

[core-ai-bot]

Comment by coelho-extremosistemas
Thursday Jul 23, 2015 at 18:02 GMT

---

Hi all
I was testing this request, but I found errors when running the brackets-shell.
Use fedora-20 x64.
The title page of this fixed "Brackets"
and after I maximixar the window, you can not write on inputs

[core-ai-bot]

Comment by ficristo
Wednesday May 25, 2016 at 08:43 GMT

---

I tryed in my Ubuntu VM and found that the close button works if I have the 3D acceleration enabled, otherwise it hangs the window.
When I hit File \ Close I always get a segfault. I tryed to debug with gdb but in the end I gave up...

@nethip What I can try to fix this?