jberger / Mojo-Chrome

A Mojo interface to Chrome DevTools Protocol
Other
10 stars 5 forks source link

add windows 10 support to detect_chrome_executable #7

Open dmanto opened 6 years ago

dmanto commented 6 years ago

detect_chrome_executable now will work in Windows 10 (tested with strawberry perl 5.26.1, also on cygwin with correspondig perl 5.26.1)

jberger commented 6 years ago

oh I like the notion of adding appveyor to this in the same patch, that would go a long way to proving that functionality to me while I cannot test it myself (mac/linux guy). I should also add travis while we're at it.

dmanto commented 6 years ago

ok now is passing tests at appveyor, but

- I don't get any warning running the tests locally on a win10 machine (not even the "DevTools listening on..." lines)

- I also get that gpu error from chrome when I run on Ubuntu 16.04.2 LTS, (tests also pass):

daniel@localhost:~/Mojo-Chrome$ prove -l t/basic.t ..... [0319/222930.323742:ERROR:gpu_process_transport_factory.cc(1019)] Lost UI shared context.

DevTools listening on ws://127.0.0.1:36071/devtools/browser/d3c9a8bf-5a38-4047-a3d8-f666c679338b t/basic.t ..... ok t/from_url.t .. ok t/tester.t .... [0319/222931.907437:ERROR:gpu_process_transport_factory.cc(1019)] Lost UI shared context.

DevTools listening on ws://127.0.0.1:48977/devtools/browser/59ced763-570d-4109-9b12-1beceb5f0abb t/tester.t .... ok All tests successful. Files=3, Tests=21, 3 wallclock secs ( 0.04 usr 0.01 sys + 1.90 cusr 0.25 csys = 2.20 CPU) Result: PASS daniel@localhost:~/Mojo-Chrome$



I am trying to investigate why are we getting those errors, not very sure what they are. I had a lot more errors and warning until I update chrome-browser to the last version (65.0.3325.162-1)
dmanto commented 6 years ago

Sorry I didn't mean to close it

heikojansen commented 6 years ago

AFAIK there's a "--disable-gpu" flag, which is said to be needed on Windows (only) with headless Chrome. Unfortunately I cannot remember right now where I read that. Maybe that would "fix" (evade?) the errors.