Bug 51256 – Fix test failures where NULL GeolocationClient is provided

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit API

[John Knottenbelt]

We need to test for NULL m_client on GeolocationClientProxy::geolocationDestroyed() as well because GeolocationClientProxy::geolocationDestroyed() is called unconditionally from the Page destructor.

e.g. webkit_unit_tests on Win. Test SelectPopupMenuTest.ShowThenHide

Backtrace:
	WebKit::GeolocationClientProxy::geolocationDestroyed [0x01E802D3+3] (d:\google-windows-2\chromium-win-release-tests\build\webkit\chromium\src\geolocationclientproxy.cpp:58)
Backtrace:
	WebKit::GeolocationClientProxy::geolocationDestroyed [0x01E802D3+3] (d:\google-windows-2\chromium-win-release-tests\build\webkit\chromium\src\geolocationclientproxy.cpp:58)
	WebCore::GeolocationController::~GeolocationController [0x015F3062+18] (d:\google-windows-2\chromium-win-release-tests\build\webcore\page\geolocationcontroller.cpp:46)
	WebCore::Page::~Page [0x0143CAC7+711] (d:\google-windows-2\chromium-win-release-tests\build\webcore\page\page.cpp:228)
	WebKit::WebViewImpl::close [0x01E5C330+96] (d:\google-windows-2\chromium-win-release-tests\build\webkit\chromium\src\webviewimpl.cpp:937)
	testing::TestInfo::Run [0x01E4D13D+269] (d:\google-windows-2\chromium-win-release-tests\build\webkit\chromium\testing\gtest\src\gtest.cc:2322)
	testing::TestCase::Run [0x01E4D2FF+239] (d:\google-windows-2\chromium-win-release-tests\build\webkit\chromium\testing\gtest\src\gtest.cc:2423)
	testing::internal::UnitTestImpl::RunAllTests [0x01E4D683+499] (d:\google-windows-2\chromium-win-release-tests\build\webkit\chromium\testing\gtest\src\gtest.cc:4199)
	testing::UnitTest::Run [0x01E4E0C9+137] (d:\google-windows-2\chromium-win-release-tests\build\webkit\chromium\testing\gtest\src\gtest.cc:3838)
	base::TestSuite::Run [0x01DE1468+216] (d:\google-windows-2\chromium-win-release-tests\build\webkit\chromium\base\test\test_suite.cc:127)
	main [0x0133AC75+37] (d:\google-windows-2\chromium-win-release-tests\build\webkit\chromium\tests\runalltests.cpp:54)

[John Knottenbelt]

Created attachment 76884 [details]
Patch

[Jeremy Orlow]

Comment on attachment 76884 [details]
Patch

Pretty sure this is right and things are currently broken, so r=me.  Steve please take a look tho.

[Steve Block]

LGTM

[WebKit Commit Bot]

The commit-queue encountered the following flaky tests while processing attachment 76884 [details]:

http/tests/navigation/target-frame-from-window.html bug 51098 (author: [e-mail])
The commit-queue is continuing to process your patch.

[WebKit Commit Bot]

Comment on attachment 76884 [details]
Patch

Clearing flags on attachment: 76884

Committed r74274: <http://trac.webkit.org/changeset/74274>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.