wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 937204 - 4/3. Avoid stalling in xpcshell or anywhere we don't have a WindowCreator - r=ehsan

Browse Source
This commit is contained in:
Benoit Jacob 2013-11-15 11:28:47 -05:00
parent 1a9cb2d9e6
commit a5fd01d0a1
1 changed files with 29 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
widget/android/GfxInfo.cpp
View File

@ -12,6 +12,8 @@
#include "nsVersionComparator.h"
#include "mozilla/Monitor.h"
#include "AndroidBridge.h"
#include "nsIWindowWatcher.h"
#include "nsServiceManagerUtils.h"
#if defined(MOZ_CRASHREPORTER)
#include "nsExceptionHandler.h"
@ -22,7 +24,26 @@
namespace mozilla {
namespace widget {
class GfxInfo::GLStrings {
static bool ExpectGLStringsToEverGetInitialized()
{
// In XPCShell, we don't have a compositor, so our GL strings will never get
// properly initialized.
// We need to know about that in GLStrings::EnsureInitialized to avoid waiting forever.
// The way we detect that we won't ever have a compositor, is that the nsWindowWatcher
// doesn't have a WindowCreator i.e. we won't create any windows. That is actually
// the root difference between xpcshell and real browsers, that causes the former
// not to create a window and a compositor.
nsCOMPtr<nsIWindowWatcher> wwatch(do_GetService(NS_WINDOWWATCHER_CONTRACTID));
if (!wwatch) {
return false;
}
bool hasWindowCreator = false;
nsresult rv = wwatch->HasWindowCreator(&hasWindowCreator);
return NS_SUCCEEDED(rv) && hasWindowCreator;
}
class GfxInfo::GLStrings
{
nsCString mVendor;
nsCString mRenderer;
nsCString mVersion;
@ -70,7 +91,13 @@ public:
MonitorAutoLock autoLock(mMonitor);
// re-check mReady, as it could have changed before we locked.
if (!mReady) {
mMonitor.Wait();
if (ExpectGLStringsToEverGetInitialized()) {
mMonitor.Wait();
} else {
// we'll never get notified, so don't wait. Just go on
// with empty GL strings.
mReady = true;
}
}
}
}