Bug 34272 – [Android] V8 Bindings are missing numerous forward-declarations and includes

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Steve Block]

In the V8 Bindings, numerous numerous forward-declarations and includes are picked up through includes guarded by feature or platform flags. This causes build errors on Android, where these flags may not be enabled.

- Scriptcontroller.h - Need to forward declare NPObject
  Chromium picks this up though V8Proxy.h -> ChromiumBridge.h
- V8DOMWrapper.h - Need to forward declare V8Proxy
  Chromium picks this up through V8CustomXpathNSResolver.h, the contents of which are guarded with XPATH
- V8DOMWindowShell.cpp - Need to include CString.h and PlatformBridge.h for Android

[Steve Block]

Created attachment 47630 [details]
Patch

[Adam Barth]

Comment on attachment 47630 [details]
Patch

Those are mostly my mistakes.  Thanks for fixing them.

How close are we to having Android buildable from svn.webkit.org?  That's the best way to catch future regressions like these.

[WebKit Commit Bot]

Comment on attachment 47630 [details]
Patch

Clearing flags on attachment: 47630

Committed r54014: <http://trac.webkit.org/changeset/54014>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Steve Block]

> How close are we to having Android buildable from svn.webkit.org?  That's the
> best way to catch future regressions like these.
We've so far made good progress with upstreaming the Android changes back to webkit.org. We're going as fast as we can and hope to be done by the end of Spring. Thanks for all your help with reviewing!