mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-10-20 08:45:46 +00:00
79aa95fa24
There's no good reason why these can't be code constants.
Especially given that, due to a bug, changes to the
"idle_queue.{min,long}_period" constants were not being passed onto the C++
code!
Here's why: those two prefs were specified as integers in all.js. But we used
AddFloatVarCache() to set up the reading of those prefs. libpref fakes floats
by storing them as strings and then converting them to floats when they are
read.
Which means that AddFloatVarCache() used to fail to get the value from all.js
-- because there's a type mismatch, int vs. string -- and instead use the
fallback default. That value is the same as the one in all.js, which is lucky.
But if someone changed the value in about:config to 100 (an integer), a similar
failure would have occured and the value used by the C++ code wouldn't be
updated!
Also note that idle_queue.max_timer_thread_bound did not have a value in
all.js.
What a mess!
--HG--
extra : rebase_source : 86f8fa905163803eb95007609c029e18c2c4f586
36 lines
846 B
C++
36 lines
846 B
C++
/* -*- Mode: C++; tab-width: 8; indent-tabs-mode: nil; c-basic-offset: 2 -*- */
|
|
/* vim: set ts=8 sts=2 et sw=2 tw=80: */
|
|
/* This Source Code Form is subject to the terms of the Mozilla Public
|
|
* License, v. 2.0. If a copy of the MPL was not distributed with this
|
|
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */
|
|
|
|
#ifndef mozilla_dom_mainthreadidleperiod_h
|
|
#define mozilla_dom_mainthreadidleperiod_h
|
|
|
|
#include "mozilla/TimeStamp.h"
|
|
#include "nsThreadUtils.h"
|
|
|
|
namespace mozilla {
|
|
|
|
class MainThreadIdlePeriod final : public IdlePeriod
|
|
{
|
|
public:
|
|
NS_DECL_NSIIDLEPERIOD
|
|
|
|
MainThreadIdlePeriod()
|
|
: mLastIdleDeadline(TimeStamp::Now())
|
|
{
|
|
}
|
|
|
|
static float GetLongIdlePeriod();
|
|
|
|
private:
|
|
virtual ~MainThreadIdlePeriod() {}
|
|
|
|
TimeStamp mLastIdleDeadline;
|
|
};
|
|
|
|
} // namespace mozilla
|
|
|
|
#endif // mozilla_dom_mainthreadidleperiod_h
|