If the feature is off the build should still work. Instead there are compile errors: WebCore/bindings/v8/custom/V8CustomPositionCallback.cpp:68: error: no matching function for call to ‘toV8(WebCore::Geoposition*&)’ out/Release/obj/gen/webkit/bindings/V8Event.h:49: note: candidates are: v8::Handle<v8::Value> WebCore::toV8(WebCore::Event*) out/Release/obj/gen/webkit/bindings/V8Event.h:50: note: v8::Handle<v8::Value> WebCore::toV8(WTF::PassRefPtr<WebCore::Event>) out/Release/obj/gen/webkit/bindings/V8XPathNSResolver.h:47: note: v8::Handle<v8::Value> WebCore::toV8(WebCore::XPathNSResolver*) out/Release/obj/gen/webkit/bindings/V8XPathNSResolver.h:48: note: v8::Handle<v8::Value> WebCore::toV8(WTF::PassRefPtr<WebCore::XPathNSResolver>) WebCore/bindings/v8/custom/V8CustomPositionErrorCallback.cpp: In member function ‘virtual void WebCore::V8CustomPositionErrorCallback::handleEvent(WebCore::PositionError*)’: WebCore/bindings/v8/custom/V8CustomPositionErrorCallback.cpp:68: error: no matching function for call to ‘toV8(WebCore::PositionError*&)’ out/Release/obj/gen/webkit/bindings/V8Event.h:49: note: candidates are: v8::Handle<v8::Value> WebCore::toV8(WebCore::Event*) out/Release/obj/gen/webkit/bindings/V8Event.h:50: note: v8::Handle<v8::Value> WebCore::toV8(WTF::PassRefPtr<WebCore::Event>) out/Release/obj/gen/webkit/bindings/V8XPathNSResolver.h:47: note: v8::Handle<v8::Value> WebCore::toV8(WebCore::XPathNSResolver*) out/Release/obj/gen/webkit/bindings/V8XPathNSResolver.h:48: note: v8::Handle<v8::Value> WebCore::toV8(WTF::PassRefPtr<WebCore::XPathNSResolver>) make: *** [out/Release/obj.target/webcore/WebCore/bindings/v8/custom/V8CustomPositionCallback.o] Error 1
yeah, looks like all of these should be protected by #if ENABLE(GEOLOCATION) ... #endif
Created attachment 64110 [details] Patch
Comment on attachment 64110 [details] Patch Clearing flags on attachment: 64110 Committed r65199: <http://trac.webkit.org/changeset/65199>
All reviewed patches have been landed. Closing bug.