search

luobenyu / chromedevtools

Automatically exported from code.google.com/p/chromedevtools
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

NPE when trying to use getTabs(backend) from Java SDK #81

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
SDK/Debugger version:0.3.8
Eclipse version: Indigo Service Release 2
Google Chrome/V8 Embedder + version: 23.0.1271.64
[For WebKit (WIP) protocol] Backend 
version:lib.org.chromium.sdk.wipbackend.wk120709.jar
OS + version:

What steps will reproduce the problem?
1. Start chrome with chrome --remote-debugging-port=2012
2. Copy the attached Maven project and run mvn test

I can attach just fine using the Eclipse Debugger version 0.3.8.201210040400. 
But I cannot run the example code given in the docs using the SDK.

What is the expected result?

What happens instead?

NPE in the line 

browser.getTabs(backend).get(0);

java.lang.NullPointerException
    at org.chromium.sdk.internal.wip.WipBackendImpl.readHttpResponseContent(WipBackendImpl.java:128)
    at org.chromium.sdk.internal.wip.WipBackendImpl.getTabs(WipBackendImpl.java:53)
    at org.chromium.sdk.internal.wip.WipBrowserImpl.getTabs(WipBrowserImpl.java:33)
    at TestCEFWithWIP.testConnectionUsingWIP(TestCEFWithWIP.java:87)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
    at java.lang.reflect.Method.invoke(Method.java:597)
    at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:47)
    at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
    at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:44)
    at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
    at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:271)
    at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:70)
    at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:50)
    at org.junit.runners.ParentRunner$3.run(ParentRunner.java:238)
    at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:63)
    at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:236)
    at org.junit.runners.ParentRunner.access$000(ParentRunner.java:53)
    at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:229)
    at org.junit.runners.ParentRunner.run(ParentRunner.java:309)
    at org.eclipse.jdt.internal.junit4.runner.JUnit4TestReference.run(JUnit4TestReference.java:50)
    at org.eclipse.jdt.internal.junit.runner.TestExecution.run(TestExecution.java:38)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:467)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:683)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:390)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:197)

Please provide any additional information below. If possible, attach a
(reduced) test-case.

Original issue reported on code.google.com by bino...@gmail.com on 27 Dec 2012 at 7:18

Attachments:

GoogleCodeExporter commented 9 years ago 
As a quick fix, you can supply non-null WipBrowserFactory.LoggerFactory value 
in createBrowser method. The class implementation may return nulls from both 
methods.

Further investigation is planned for after holidays.

Original comment by peter.ry...@gmail.com on 1 Jan 2013 at 2:23

GoogleCodeExporter commented 9 years ago 
great, using the instance that returns null values works for now. On to the 
next step. Thanks!

Original comment by bino...@gmail.com on 2 Jan 2013 at 3:21

GoogleCodeExporter commented 9 years ago

Original comment by peter.ry...@gmail.com on 2 Jan 2013 at 8:02

GoogleCodeExporter commented 9 years ago 
Fixed in HEAD.

Original comment by peter.ry...@gmail.com on 10 Jan 2013 at 4:10