Bug 1338011 - Adds some telemetry probes - f=bsmedberg r=gerald

MozReview-Commit-ID: BmwEjBiCBNo
This commit is contained in:
Jay Harris
2017-04-15 09:36:42 +12:00
parent 315fc078db
commit 86ba6810a2
2 changed files with 26 additions and 1 deletions

View File

@@ -51,6 +51,13 @@ namespace gfx {
using namespace mozilla::layers;
enum class FallbackType : uint32_t
{
NONE = 0,
DECODINGDISABLED,
DISABLED,
};
static StaticAutoPtr<GPUProcessManager> sSingleton;
GPUProcessManager*
@@ -163,6 +170,9 @@ GPUProcessManager::DisableGPUProcess(const char* aMessage)
gfxPlatform::NotifyGPUProcessDisabled();
Telemetry::Accumulate(Telemetry::GPU_PROCESS_CRASH_FALLBACKS,
uint32_t(FallbackType::DISABLED));
DestroyProcess();
ShutdownVsyncIOThread();
}
@@ -379,8 +389,14 @@ GPUProcessManager::OnProcessUnexpectedShutdown(GPUProcessHost* aHost)
SprintfLiteral(disableMessage, "GPU process disabled after %d attempts",
mNumProcessAttempts);
DisableGPUProcess(disableMessage);
} else if (mNumProcessAttempts > uint32_t(gfxPrefs::GPUProcessMaxRestartsWithDecoder())) {
} else if (mNumProcessAttempts > uint32_t(gfxPrefs::GPUProcessMaxRestartsWithDecoder()) &&
mDecodeVideoOnGpuProcess) {
mDecodeVideoOnGpuProcess = false;
Telemetry::Accumulate(Telemetry::GPU_PROCESS_CRASH_FALLBACKS,
uint32_t(FallbackType::DECODINGDISABLED));
} else {
Telemetry::Accumulate(Telemetry::GPU_PROCESS_CRASH_FALLBACKS,
uint32_t(FallbackType::NONE));
}
HandleProcessLost();