Skip to content
This repository has been archived by the owner on Aug 4, 2022. It is now read-only.

Commit

Permalink
Bug 1304829 - rename RELEASE_BUILD to RELEASE_OR_BETA: main part. r=t…
Browse files Browse the repository at this point in the history
…ed,Mossop

MozReview-Commit-ID: 1lCt0xTMV5O
  • Loading branch information
Archaeopteryx committed Oct 8, 2016
1 parent 0734ecf commit 5ed972d
Show file tree
Hide file tree
Showing 79 changed files with 163 additions and 166 deletions.
2 changes: 1 addition & 1 deletion b2g/app/b2g.js
Original file line number Diff line number Diff line change
Expand Up @@ -1011,7 +1011,7 @@ pref("dom.activities.developer_mode_only", "import-app");
pref("dom.serviceWorkers.enabled", false);
pref("dom.push.enabled", false);

#if defined(RELEASE_BUILD)
#if defined(RELEASE_OR_BETA)
// Bug 1278848: Enable service worker notifications on release B2G once
// they're ready.
pref("dom.webnotifications.serviceworker.enabled", false);
Expand Down
16 changes: 8 additions & 8 deletions browser/app/profile/firefox.js
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ pref("app.update.auto", true);
pref("app.update.silent", false);

// If set to true, the hamburger button will show badges for update events.
#ifndef RELEASE_BUILD
#ifndef RELEASE_OR_BETA
pref("app.update.badge", true);
#else
pref("app.update.badge", false);
Expand Down Expand Up @@ -328,7 +328,7 @@ pref("browser.download.folderList", 1);
pref("browser.download.manager.addToRecentDocs", true);
pref("browser.download.manager.resumeOnWakeDelay", 10000);

#ifdef RELEASE_BUILD
#ifdef RELEASE_OR_BETA
pref("browser.download.showPanelDropmarker", false);
#else
pref("browser.download.showPanelDropmarker", true);
Expand Down Expand Up @@ -1244,14 +1244,14 @@ pref("plain_text.wrap_long_lines", true);
pref("dom.debug.propagate_gesture_events_through_content", false);

// The request URL of the GeoLocation backend.
#ifdef RELEASE_BUILD
#ifdef RELEASE_OR_BETA
pref("geo.wifi.uri", "https://www.googleapis.com/geolocation/v1/geolocate?key=%GOOGLE_API_KEY%");
#else
pref("geo.wifi.uri", "https://location.services.mozilla.com/v1/geolocate?key=%MOZILLA_API_KEY%");
#endif

#ifdef XP_MACOSX
#ifdef RELEASE_BUILD
#ifdef RELEASE_OR_BETA
pref("geo.provider.use_corelocation", false);
#else
pref("geo.provider.use_corelocation", true);
Expand All @@ -1264,7 +1264,7 @@ pref("geo.provider.ms-windows-location", false);

#ifdef MOZ_WIDGET_GTK
#ifdef MOZ_GPSD
#ifdef RELEASE_BUILD
#ifdef RELEASE_OR_BETA
pref("geo.provider.use_gpsd", false);
#else
pref("geo.provider.use_gpsd", true);
Expand Down Expand Up @@ -1428,7 +1428,7 @@ pref("privacy.userContext.enabled", false);
pref("privacy.userContext.ui.enabled", false);
#endif

#ifndef RELEASE_BUILD
#ifndef RELEASE_OR_BETA
// At the moment, autostart.2 is used, while autostart.1 is unused.
// We leave it here set to false to reset users' defaults and allow
// us to change everybody to true in the future, when desired.
Expand All @@ -1447,7 +1447,7 @@ pref("extensions.interposition.enabled", true);
pref("extensions.interposition.prefetching", true);

// Enable blocking of e10s for add-on users on beta/release.
#ifdef RELEASE_BUILD
#ifdef RELEASE_OR_BETA
pref("extensions.e10sBlocksEnabling", true);
#endif

Expand Down Expand Up @@ -1530,7 +1530,7 @@ pref("webchannel.allowObject.urlWhitelist", "https://accounts.firefox.com https:
// Whether or not the browser should scan for unsubmitted
// crash reports, and then show a notification for submitting
// those reports.
#ifdef RELEASE_BUILD
#ifdef RELEASE_OR_BETA
pref("browser.crashReports.unsubmittedCheck.enabled", false);
#else
pref("browser.crashReports.unsubmittedCheck.enabled", true);
Expand Down
2 changes: 1 addition & 1 deletion browser/base/content/browser-devedition.js
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ var DevEdition = {
// If the DevEdition theme is going to be applied in gBrowserInit.onLoad,
// then preload it now. This prevents a flash of unstyled content where the
// normal theme is applied while the DevEdition stylesheet is loading.
if (!AppConstants.RELEASE_BUILD &&
if (!AppConstants.RELEASE_OR_BETA &&
this != Services.appShell.hiddenDOMWindow && DevEdition.isThemeCurrentlyApplied) {
DevEdition.createStyleSheet();
}
6 changes: 3 additions & 3 deletions browser/components/nsBrowserGlue.js
Original file line number Diff line number Diff line change
Expand Up @@ -703,7 +703,7 @@ BrowserGlue.prototype = {
// Ensure we keep track of places/pw-mananager undo by init'ing this early.
Cu.import("resource:///modules/AutoMigrate.jsm");

if (!AppConstants.RELEASE_BUILD) {
if (!AppConstants.RELEASE_OR_BETA) {
let themeName = gBrowserBundle.GetStringFromName("deveditionTheme.name");
let vendorShortName = gBrandBundle.GetStringFromName("vendorShortName");

Expand Down Expand Up @@ -1150,7 +1150,7 @@ BrowserGlue.prototype = {
ShellService.shouldCheckDefaultBrowser;
let promptCount;
let skipDefaultBrowserCheck = false;
if (!AppConstants.RELEASE_BUILD) {
if (!AppConstants.RELEASE_OR_BETA) {
promptCount =
Services.prefs.getIntPref("browser.shell.defaultBrowserCheckCount");
skipDefaultBrowserCheck =
Expand Down Expand Up @@ -1195,7 +1195,7 @@ BrowserGlue.prototype = {
}
}

if (!AppConstants.RELEASE_BUILD) {
if (!AppConstants.RELEASE_OR_BETA) {
if (willPrompt) {
Services.prefs.setIntPref("browser.shell.defaultBrowserCheckCount",
promptCount);
Expand Down
4 changes: 2 additions & 2 deletions build/docs/mozinfo.rst
Original file line number Diff line number Diff line change
Expand Up @@ -129,8 +129,8 @@ processor

Always defined.

release_build
Whether this is a release build.
release_or_beta
Whether this is a release or beta build.

Values are ``true`` and ``false``.

Expand Down
18 changes: 9 additions & 9 deletions build/moz.configure/init.configure
Original file line number Diff line number Diff line change
Expand Up @@ -663,11 +663,11 @@ set_define('MOZ_BUILD_APP', build_project)
add_old_configure_assignment('MOZ_BUILD_APP', build_project)


# set RELEASE_BUILD and NIGHTLY_BUILD variables depending on the cycle we're in
# set RELEASE_OR_BETA and NIGHTLY_BUILD variables depending on the cycle we're in
# The logic works like this:
# - if we have "a1" in GRE_MILESTONE, we're building Nightly (define NIGHTLY_BUILD)
# - otherwise, if we have "a" in GRE_MILESTONE, we're building Nightly or Aurora
# - otherwise, we're building Release/Beta (define RELEASE_BUILD)
# - otherwise, we're building Release/Beta (define RELEASE_OR_BETA)
@depends(check_build_environment, '--help')
@imports(_from='__builtin__', _import='open')
def milestone(build_env, _):
Expand All @@ -677,26 +677,26 @@ def milestone(build_env, _):
with open(milestone_path, 'r') as fh:
milestone = fh.read().splitlines()[-1]

is_nightly = is_release = None
is_nightly = is_release_or_beta = None

if 'a1' in milestone:
is_nightly = True
elif 'a' not in milestone:
is_release = True
is_release_or_beta = True

return namespace(version=milestone,
is_nightly=is_nightly,
is_release=is_release)
is_release_or_beta=is_release_or_beta)

set_config('GRE_MILESTONE', delayed_getattr(milestone, 'version'))
set_config('NIGHTLY_BUILD', delayed_getattr(milestone, 'is_nightly'))
set_define('NIGHTLY_BUILD', delayed_getattr(milestone, 'is_nightly'))
add_old_configure_assignment('NIGHTLY_BUILD',
delayed_getattr(milestone, 'is_nightly'))
set_config('RELEASE_BUILD', delayed_getattr(milestone, 'is_release'))
set_define('RELEASE_BUILD', delayed_getattr(milestone, 'is_release'))
add_old_configure_assignment('RELEASE_BUILD',
delayed_getattr(milestone, 'is_release'))
set_config('RELEASE_OR_BETA', delayed_getattr(milestone, 'is_release_or_beta'))
set_define('RELEASE_OR_BETA', delayed_getattr(milestone, 'is_release_or_beta'))
add_old_configure_assignment('RELEASE_OR_BETA',
delayed_getattr(milestone, 'is_release_or_beta'))

# The app update channel is 'default' when not supplied. The value is used in
# the application's confvars.sh (and is made available to a project specific
Expand Down
2 changes: 1 addition & 1 deletion build/release/sanity.py
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ def sendchange(branch, revision, username, master, products):
'products:%s' % ','.join(products),
'-p',
'script_repo_revision:%s' % revision,
'release_build'
'release_or_beta'
]
logging.info("Executing: %s" % cmd)
run_cmd(cmd)
Expand Down
2 changes: 1 addition & 1 deletion devtools/client/preferences/devtools.js
Original file line number Diff line number Diff line change
Expand Up @@ -357,7 +357,7 @@ pref("devtools.telemetry.tools.opened.version", "{}");

// Enable the JSON View tool (an inspector for application/json documents) on
// Nightly and Dev. Edition.
#ifdef RELEASE_BUILD
#ifdef RELEASE_OR_BETA
pref("devtools.jsonview.enabled", false);
#else
pref("devtools.jsonview.enabled", true);
Expand Down
2 changes: 1 addition & 1 deletion dom/base/nsDOMClassInfo.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1701,7 +1701,7 @@ nsWindowSH::NameStructEnabled(JSContext* aCx, nsGlobalWindow *aWin,
OldBindingConstructorEnabled(nameStruct, aWin, aCx);
}

#ifdef RELEASE_BUILD
#ifdef RELEASE_OR_BETA
#define USE_CONTROLLERS_SHIM
#endif

Expand Down
2 changes: 1 addition & 1 deletion dom/base/nsGlobalWindow.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -860,7 +860,7 @@ nsOuterWindowProxy::defineProperty(JSContext* cx,
return result.failCantDefineWindowElement();
}

#ifndef RELEASE_BUILD // To be turned on in bug 1178638.
#ifndef RELEASE_OR_BETA // To be turned on in bug 1178638.
// For now, allow chrome code to define non-configurable properties
// on windows, until we sort out what exactly the addon SDK is
// doing. In the meantime, this still allows us to test web compat
Expand Down
2 changes: 1 addition & 1 deletion dom/base/test/mochitest.ini
Original file line number Diff line number Diff line change
Expand Up @@ -825,7 +825,7 @@ skip-if = buildapp == 'b2g' || toolkit == 'android' # TC: Bug 1144079 - Re-enabl
[test_window_constructor.html]
[test_window_cross_origin_props.html]
[test_window_define_nonconfigurable.html]
skip-if = release_build
skip-if = release_or_beta
[test_window_define_symbol.html]
[test_window_element_enumeration.html]
[test_window_enumeration.html]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -46,12 +46,12 @@
if (canBeUnprivileged) {
ok(exts.indexOf("WEBGL_debug_renderer_info") != -1,
"WEBGL_debug_renderer_info should be listed by getSupportedExtensions in"
+ " non-chrome contexts on non-RELEASE_BUILDs");
+ " non-chrome contexts on non-RELEASE_OR_BETAs");

var ext = gl.getExtension("WEBGL_debug_renderer_info");
ok(!!ext,
"WEBGL_debug_renderer_info should be available through getExtension in non-chrome"
+ " contexts on non-RELEASE_BUILDs");
+ " contexts on non-RELEASE_OR_BETAs");

ok(gl.getParameter(UNMASKED_VENDOR_WEBGL) && gl.getError() == gl.NO_ERROR,
"Should be able to query UNMASKED_VENDOR_WEBGL if enabling"
Expand Down
6 changes: 3 additions & 3 deletions dom/canvas/test/chrome/test_webgl_debug_renderer_info.html
Original file line number Diff line number Diff line change
Expand Up @@ -21,10 +21,10 @@
var Cu = parent.Components.utils;
Cu.import("resource://gre/modules/AppConstants.jsm");
// This gives us `AppConstants` in the global scope.
// We need this because we only expose debug_renderer_info #ifndef MOZ_RELEASE_BUILD.
// This should match AppConstants.RELEASE_BUILD.
// We need this because we only expose debug_renderer_info #ifndef RELEASE_OR_BETA.
// This should match AppConstants.RELEASE_OR_BETA.

const canBeUnprivileged = !AppConstants.RELEASE_BUILD;
const canBeUnprivileged = !AppConstants.RELEASE_OR_BETA;


function isNonEmptyString(s)
Expand Down
2 changes: 1 addition & 1 deletion dom/media/MediaPrefs.h
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,7 @@ class MediaPrefs final
DECL_MEDIA_PREF("media.ogg.flac.enabled", FlacInOgg, bool, false);
DECL_MEDIA_PREF("media.flac.enabled", FlacEnabled, bool, true);

#if defined(MOZ_RUST_MP4PARSE) && !defined(RELEASE_BUILD)
#if defined(MOZ_RUST_MP4PARSE) && !defined(RELEASE_OR_BETA)
DECL_MEDIA_PREF("media.rust.test_mode", RustTestMode, bool, false);
#endif

Expand Down
2 changes: 1 addition & 1 deletion dom/plugins/base/nsPluginTags.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -375,7 +375,7 @@ void nsPluginTag::InitMime(const char* const* aMimeTypes,
break;
case nsPluginHost::eSpecialType_None:
default:
#ifndef RELEASE_BUILD
#ifndef RELEASE_OR_BETA
// Allow async init for all plugins on Nightly and Aurora
mSupportsAsyncInit = true;
#endif
Expand Down
2 changes: 1 addition & 1 deletion dom/plugins/test/mochitest/mochitest.ini
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ skip-if = !crashreporter || e10s
skip-if = !crashreporter || e10s
[test_hidden_plugin.html]
[test_instance_re-parent.html]
skip-if = release_build # Bug 1172627
skip-if = release_or_beta # Bug 1172627
[test_instance_unparent1.html]
[test_instance_unparent2.html]
[test_instance_unparent3.html]
Expand Down
10 changes: 5 additions & 5 deletions gfx/thebes/gfxFont.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,7 @@ gfxFontCache::gfxFontCache()
obs->AddObserver(new Observer, "memory-pressure", false);
}

#ifndef RELEASE_BUILD
#ifndef RELEASE_OR_BETA
// Currently disabled for release builds, due to unexplained crashes
// during expiration; see bug 717175 & 894798.
mWordCacheExpirationTimer = do_CreateInstance("@mozilla.org/timer;1");
Expand Down Expand Up @@ -2580,7 +2580,7 @@ gfxFont::GetShapedWord(DrawTarget *aDrawTarget,
Telemetry::Accumulate((isContent ? Telemetry::WORD_CACHE_HITS_CONTENT :
Telemetry::WORD_CACHE_HITS_CHROME),
aLength);
#ifndef RELEASE_BUILD
#ifndef RELEASE_OR_BETA
if (aTextPerf) {
aTextPerf->current.wordCacheHit++;
}
Expand All @@ -2591,7 +2591,7 @@ gfxFont::GetShapedWord(DrawTarget *aDrawTarget,
Telemetry::Accumulate((isContent ? Telemetry::WORD_CACHE_MISSES_CONTENT :
Telemetry::WORD_CACHE_MISSES_CHROME),
aLength);
#ifndef RELEASE_BUILD
#ifndef RELEASE_OR_BETA
if (aTextPerf) {
aTextPerf->current.wordCacheMiss++;
}
Expand Down Expand Up @@ -2855,7 +2855,7 @@ gfxFont::ShapeTextWithoutWordCache(DrawTarget *aDrawTarget,
return ok;
}

#ifndef RELEASE_BUILD
#ifndef RELEASE_OR_BETA
#define TEXT_PERF_INCR(tp, m) (tp ? (tp)->current.m++ : 0)
#else
#define TEXT_PERF_INCR(tp, m)
Expand Down Expand Up @@ -2895,7 +2895,7 @@ gfxFont::SplitAndInitTextRun(DrawTarget *aDrawTarget,

gfxTextPerfMetrics *tp = nullptr;

#ifndef RELEASE_BUILD
#ifndef RELEASE_OR_BETA
tp = aTextRun->GetFontGroup()->GetTextPerfMetrics();
if (tp) {
if (mStyle.systemFont) {
Expand Down
2 changes: 1 addition & 1 deletion gfx/thebes/gfxPlatform.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -390,7 +390,7 @@ NS_IMPL_ISUPPORTS_INHERITED0(CrashTelemetryEvent, Runnable);
void
CrashStatsLogForwarder::CrashAction(LogReason aReason)
{
#ifndef RELEASE_BUILD
#ifndef RELEASE_OR_BETA
// Non-release builds crash by default, but will use telemetry
// if this environment variable is present.
static bool useTelemetry = gfxEnv::GfxDevCrashTelemetry();
Expand Down
2 changes: 1 addition & 1 deletion gfx/thebes/gfxPrefs.h
Original file line number Diff line number Diff line change
Expand Up @@ -343,7 +343,7 @@ class gfxPrefs final
#if defined(ANDROID)
DECL_GFX_PREF(Once, "gfx.apitrace.enabled", UseApitrace, bool, false);
#endif
#if defined(RELEASE_BUILD)
#if defined(RELEASE_OR_BETA)
// "Skip" means this is locked to the default value in beta and release.
DECL_GFX_PREF(Skip, "gfx.blocklist.all", BlocklistAll, int32_t, 0);
#else
Expand Down
8 changes: 4 additions & 4 deletions gfx/thebes/gfxTextRun.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ gfxTextRun::gfxTextRun(const gfxTextRunFactory::Parameters *aParams,
MOZ_COUNT_CTOR(gfxTextRun);
NS_ADDREF(mFontGroup);

#ifndef RELEASE_BUILD
#ifndef RELEASE_OR_BETA
gfxTextPerfMetrics *tp = aFontGroup->GetTextPerfMetrics();
if (tp) {
tp->current.textrunConst++;
Expand Down Expand Up @@ -191,7 +191,7 @@ gfxTextRun::~gfxTextRun()
// been told to release its reference to the group, so we mustn't do that
// again here.
if (!mReleasedFontGroup) {
#ifndef RELEASE_BUILD
#ifndef RELEASE_OR_BETA
gfxTextPerfMetrics *tp = mFontGroup->GetTextPerfMetrics();
if (tp) {
tp->current.textrunDestr++;
Expand Down Expand Up @@ -2902,7 +2902,7 @@ void gfxFontGroup::ComputeRanges(nsTArray<gfxTextRange>& aRanges,
FindFontForChar(ch, prevCh, nextCh, aRunScript, prevFont,
&matchType);

#ifndef RELEASE_BUILD
#ifndef RELEASE_OR_BETA
if (MOZ_UNLIKELY(mTextPerf)) {
if (matchType == gfxTextRange::kPrefsFallback) {
mTextPerf->current.fallbackPrefs++;
Expand Down Expand Up @@ -2961,7 +2961,7 @@ void gfxFontGroup::ComputeRanges(nsTArray<gfxTextRange>& aRanges,

aRanges[lastRangeIndex].end = aLength;

#ifndef RELEASE_BUILD
#ifndef RELEASE_OR_BETA
LogModule* log = mStyle.systemFont
? gfxPlatform::GetLog(eGfxLog_textrunui)
: gfxPlatform::GetLog(eGfxLog_textrun);
Expand Down
2 changes: 1 addition & 1 deletion gfx/thebes/gfxUserFontSet.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ class gfxOTSContext : public ots::OTSContext {
virtual ots::TableAction GetTableAction(uint32_t aTag) override {
// Preserve Graphite, color glyph and SVG tables
if (
#ifdef RELEASE_BUILD // For Beta/Release, also allow OT Layout tables through
#ifdef RELEASE_OR_BETA // For Beta/Release, also allow OT Layout tables through
// unchecked, and rely on harfbuzz to handle them safely.
aTag == TRUETYPE_TAG('G', 'D', 'E', 'F') ||
aTag == TRUETYPE_TAG('G', 'P', 'O', 'S') ||
Expand Down
2 changes: 1 addition & 1 deletion ipc/chromium/src/base/pickle.h
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
#include "mozilla/ipc/Faulty.h"
#endif

#if !defined(RELEASE_BUILD) || defined(DEBUG)
#if !defined(RELEASE_OR_BETA) || defined(DEBUG)
#define MOZ_PICKLE_SENTINEL_CHECKING
#endif

Expand Down
Loading

0 comments on commit 5ed972d

Please sign in to comment.