Bug 1348820 - Remove MOZ_ANDROID_ACTIVITY_STREAM build flag. r=Grisha

This flag wasn't used in the build system anyways. It was used to hide early builds of the
new Activity Stream UI. However this is now going to be controlled by Switchboard experiments.

MozReview-Commit-ID: Dfzw9YGgHkN

--HG--
extra : rebase_source : 2ecd3fb18a237f0de8e7d4752d69de568c062310
This commit is contained in:
Sebastian Kaspari 2017-03-20 14:31:33 +01:00
Родитель f6623be0e7
Коммит 074e92e4fe
8 изменённых файлов: 3 добавлений и 32 удалений

Просмотреть файл

@ -318,11 +318,4 @@ public class AppConstants {
// (bug 1266820) Temporarily disabled since no one is working on it. // (bug 1266820) Temporarily disabled since no one is working on it.
public static final boolean SCREENSHOTS_IN_BOOKMARKS_ENABLED = false; public static final boolean SCREENSHOTS_IN_BOOKMARKS_ENABLED = false;
public static final boolean MOZ_ANDROID_ACTIVITY_STREAM =
//#ifdef MOZ_ANDROID_ACTIVITY_STREAM
true;
//#else
false;
//#endif
} }

Просмотреть файл

@ -37,8 +37,7 @@ def main(output_file, input_filename):
CONFIG.update(buildconfig.substs) CONFIG.update(buildconfig.substs)
DEFINES = {} DEFINES = {}
for var in ('MOZ_ANDROID_ACTIVITY_STREAM' for var in ('MOZ_ANDROID_ANR_REPORTER',
'MOZ_ANDROID_ANR_REPORTER',
'MOZ_ANDROID_BEAM', 'MOZ_ANDROID_BEAM',
'MOZ_ANDROID_CUSTOM_TABS', 'MOZ_ANDROID_CUSTOM_TABS',
'MOZ_ANDROID_DOWNLOADS_INTEGRATION', 'MOZ_ANDROID_DOWNLOADS_INTEGRATION',

Просмотреть файл

@ -63,11 +63,6 @@ public class ActivityStream {
* Is the user eligible to use activity stream or should we hide it from settings etc.? * Is the user eligible to use activity stream or should we hide it from settings etc.?
*/ */
public static boolean isUserEligible(Context context) { public static boolean isUserEligible(Context context) {
if (AppConstants.MOZ_ANDROID_ACTIVITY_STREAM) {
// If the build flag is enabled then just show the option to the user.
return true;
}
if (AppConstants.NIGHTLY_BUILD && SwitchBoard.isInExperiment(context, Experiments.ACTIVITY_STREAM)) { if (AppConstants.NIGHTLY_BUILD && SwitchBoard.isInExperiment(context, Experiments.ACTIVITY_STREAM)) {
// If this is a nightly build and the user is part of the activity stream experiment then // If this is a nightly build and the user is part of the activity stream experiment then
// the option should be visible too. The experiment is limited to Nightly too but I want // the option should be visible too. The experiment is limited to Nightly too but I want

Просмотреть файл

@ -714,7 +714,6 @@ public class GeckoPreferences
i--; i--;
continue; continue;
} else if (PREFS_CATEGORY_EXPERIMENTAL_FEATURES.equals(key) } else if (PREFS_CATEGORY_EXPERIMENTAL_FEATURES.equals(key)
&& !AppConstants.MOZ_ANDROID_ACTIVITY_STREAM
&& !AppConstants.MOZ_ANDROID_CUSTOM_TABS) { && !AppConstants.MOZ_ANDROID_CUSTOM_TABS) {
preferences.removePreference(pref); preferences.removePreference(pref);
i--; i--;

Просмотреть файл

@ -1123,8 +1123,7 @@ for var in ('MOZ_ANDROID_ANR_REPORTER', 'MOZ_DEBUG',
'MOZ_ANDROID_DOWNLOADS_INTEGRATION', 'MOZ_INSTALL_TRACKING', 'MOZ_ANDROID_DOWNLOADS_INTEGRATION', 'MOZ_INSTALL_TRACKING',
'MOZ_ANDROID_GCM', 'MOZ_ANDROID_EXCLUDE_FONTS', 'MOZ_LOCALE_SWITCHER', 'MOZ_ANDROID_GCM', 'MOZ_ANDROID_EXCLUDE_FONTS', 'MOZ_LOCALE_SWITCHER',
'MOZ_ANDROID_BEAM', 'MOZ_ANDROID_DOWNLOAD_CONTENT_SERVICE', 'MOZ_ANDROID_BEAM', 'MOZ_ANDROID_DOWNLOAD_CONTENT_SERVICE',
'MOZ_SWITCHBOARD', 'MOZ_ANDROID_CUSTOM_TABS', 'MOZ_SWITCHBOARD', 'MOZ_ANDROID_CUSTOM_TABS'):
'MOZ_ANDROID_ACTIVITY_STREAM'):
if CONFIG[var]: if CONFIG[var]:
DEFINES[var] = 1 DEFINES[var] = 1

Просмотреть файл

@ -4080,7 +4080,7 @@ Tab.prototype = {
this.browser.addEventListener("pagehide", listener, true); this.browser.addEventListener("pagehide", listener, true);
} }
if (AppConstants.NIGHTLY_BUILD || AppConstants.MOZ_ANDROID_ACTIVITY_STREAM) { if (AppConstants.NIGHTLY_BUILD) {
if (!docURI.startsWith("about:")) { if (!docURI.startsWith("about:")) {
WebsiteMetadata.parseAsynchronously(this.browser.contentDocument); WebsiteMetadata.parseAsynchronously(this.browser.contentDocument);
} }

Просмотреть файл

@ -48,13 +48,6 @@ project_flag('MOZ_SWITCHBOARD',
help='Include Switchboard A/B framework on Android', help='Include Switchboard A/B framework on Android',
default=True) default=True)
option(env='MOZ_ANDROID_ACTIVITY_STREAM',
help='Enable Activity Stream on Android (replacing the default HomePager)',
default=False)
set_config('MOZ_ANDROID_ACTIVITY_STREAM',
depends_if('MOZ_ANDROID_ACTIVITY_STREAM')(lambda _: True))
option(env='MOZ_ANDROID_PACKAGE_INSTALL_BOUNCER', option(env='MOZ_ANDROID_PACKAGE_INSTALL_BOUNCER',
help='Build and package the install bouncer APK', help='Build and package the install bouncer APK',
default=True) default=True)

Просмотреть файл

@ -302,13 +302,6 @@ this.AppConstants = Object.freeze({
false, false,
#endif #endif
MOZ_ANDROID_ACTIVITY_STREAM:
#ifdef MOZ_ANDROID_ACTIVITY_STREAM
true,
#else
false,
#endif
DLL_PREFIX: "@DLL_PREFIX@", DLL_PREFIX: "@DLL_PREFIX@",
DLL_SUFFIX: "@DLL_SUFFIX@", DLL_SUFFIX: "@DLL_SUFFIX@",