Closed Bug 1386975 Opened 7 years ago Closed 7 years ago

(photon) Enable Photon by default

Categories

(Firefox for Android Graveyard :: General, enhancement)

Product:
Firefox for Android Graveyard
Component:
General
Platform:
Unspecified
Android
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

(firefox57 verified)

Status:
VERIFIED FIXED
Milestone:
Firefox 57
Tracking Flags:
Tracking Status
firefox57 --- verified

People

(Reporter: jwu, Assigned: jwu)

References

Details

(Whiteboard: [FNC][SPT57.1][INT])

Attachments

(2 files)

Bug 1386975 - Part 1: Use Photon resources to build Fennec.
59 bytes, text/x-review-board-request
maliu
: review+
nalexander
: review+
sebastian
: review+
Details
Bug 1386975 - Part 2: Fix Robocop/Mochitest-chrome errors when Photon is enabled.
59 bytes, text/x-review-board-request
maliu
: review+
nalexander
: review+
sebastian
: review+
Details 
Photon is targeted on 57, it's time to enable it by default on Nightly.
Comment on attachment 8893727 [details]
Bug 1386975 - Part 1: Use Photon resources to build Fennec.

https://reviewboard.mozilla.org/r/164846/#review170248
Attachment #8893727 - Flags: review?(s.kaspari) → review+
Comment on attachment 8893728 [details]
Bug 1386975 - Part 2: Fix Robocop/Mochitest-chrome errors when Photon is enabled.

https://reviewboard.mozilla.org/r/164848/#review170250
Attachment #8893728 - Flags: review?(s.kaspari) → review+
Comment on attachment 8893727 [details]
Bug 1386975 - Part 1: Use Photon resources to build Fennec.

https://reviewboard.mozilla.org/r/164846/#review170740
Attachment #8893727 - Flags: review?(max) → review+
Comment on attachment 8893728 [details]
Bug 1386975 - Part 2: Fix Robocop/Mochitest-chrome errors when Photon is enabled.

https://reviewboard.mozilla.org/r/164848/#review170742
Attachment #8893728 - Flags: review?(max) → review+
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/88017de7130d
Part 1: Use Photon resources to build Fennec. r=maliu,sebastian
https://hg.mozilla.org/integration/autoland/rev/bd83328c7773
Part 2: Fix Robocop errors when Photon is enabled. r=maliu,sebastian
Keywords: checkin-needed
Backed out for failing chrome tests on Android 4.2 x86 opt:

https://hg.mozilla.org/integration/autoland/rev/df30da8bc862d763d715b405cca23d398ee5312e
https://hg.mozilla.org/integration/autoland/rev/0701ae812e9f47a24c666dfe3aff046e480381c4

The screenshot shows a crash.

Push with failure: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=bd83328c7773af9ed1c22592532c3d8da9b2a1a5&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-resultStatus=retry&filter-resultStatus=runnable
Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=121445805&repo=autoland

[task 2017-08-07T16:18:08.729418Z] 16:18:08     INFO -  pushing /home/worker/workspace/build/tests/mochitest/chrome to /mnt/sdcard/tests/chrome on device...
[task 2017-08-07T16:19:45.927950Z] 16:19:45     INFO -  runtests.py | Failed to copy /home/worker/workspace/build/tests/mochitest/hyphenation to profile
[task 2017-08-07T16:19:46.422746Z] 16:19:46     INFO -  pk12util: PKCS12 IMPORT SUCCESSFUL
[task 2017-08-07T16:19:46.936763Z] 16:19:46     INFO -  MochitestServer : launching [u'/home/worker/workspace/build/hostutils/host-utils-50.0a1.en-US.linux-x86_64/xpcshell', '-g', '/home/worker/workspace/build/hostutils/host-utils-50.0a1.en-US.linux-x86_64', '-v', '170', '-f', '/home/worker/workspace/build/hostutils/host-utils-50.0a1.en-US.linux-x86_64/components/httpd.js', '-e', "const _PROFILE_PATH = '/tmp/tmpI8XSgj.mozrunner'; const _SERVER_PORT = '8854'; const _SERVER_ADDR = '10.0.2.2'; const _TEST_PREFIX = undefined; const _DISPLAY_RESULTS = false;", '-f', '/home/worker/workspace/build/tests/mochitest/server.js']
[task 2017-08-07T16:19:46.937023Z] 16:19:46     INFO -  runtests.py | Server pid: 1061
[task 2017-08-07T16:19:46.942001Z] 16:19:46     INFO -  runtests.py | Websocket server pid: 1064
[task 2017-08-07T16:19:46.955178Z] 16:19:46     INFO -  runtests.py | SSL tunnel pid: 1067
[task 2017-08-07T16:19:52.423059Z] 16:19:52     INFO -  runtests.py | Running with e10s: False
[task 2017-08-07T16:19:52.423522Z] 16:19:52     INFO -  runtests.py | Running tests: start.
[task 2017-08-07T16:20:02.221668Z] 16:20:02     INFO -  INFO | automation.py | Application pid: 1911
[task 2017-08-07T16:26:04.075489Z] 16:26:04     INFO -  TEST-INFO | started process screentopng
[task 2017-08-07T16:26:04.408099Z] 16:26:04     INFO -  TEST-INFO | screentopng: exit 0
[task 2017-08-07T16:26:10.510125Z] 16:26:10  WARNING -  TEST-UNEXPECTED-FAIL | remoteautomation.py | application timed out after 330 seconds with no output
Flags: needinfo?(topwu.tw)
Comment on attachment 8893727 [details]
Bug 1386975 - Part 1: Use Photon resources to build Fennec.

https://reviewboard.mozilla.org/r/164846/#review170832

lgtm.  I'll defer to others for the rest of this process though -- I'm on parental leave and shouldn't block anybody.
Attachment #8893727 - Flags: review?(nalexander) → review+
Comment on attachment 8893728 [details]
Bug 1386975 - Part 2: Fix Robocop/Mochitest-chrome errors when Photon is enabled.

https://reviewboard.mozilla.org/r/164848/#review170834
Attachment #8893728 - Flags: review?(nalexander) → review+
The previous patches didn't consider mochitest chrome test, here is the result with new patches:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=9fc9459d5782043100ef71c58b4d5431e27414e1
Flags: needinfo?(topwu.tw)
Keywords: checkin-needed
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/64adea19eb04
Part 1: Use Photon resources to build Fennec. r=maliu,nalexander,sebastian
https://hg.mozilla.org/integration/autoland/rev/6df7dff00b94
Part 2: Fix Robocop/Mochitest-chrome errors when Photon is enabled. r=maliu,nalexander,sebastian
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/64adea19eb04
https://hg.mozilla.org/mozilla-central/rev/6df7dff00b94
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox57: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Depends on: 1388417
Depends on: 1388490
Whiteboard: [FNC][SPT57.1][INT]
Photon UI is now the default on Nightly 57. Verified as fixed.
Status: RESOLVED → VERIFIED
status-firefox57: fixed → verified
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: