Kan-Ru Chen [:kanru] (UTC+9) Reporter
	Description • 9 years ago

STR:

 1. Having someone call you
 2. Answer the call
 3. Hang up

Expected result:
 No missed call

Actual result:
 Missed call notification, on the lock screen and in the call log

	Kan-Ru Chen [:kanru] (UTC+9) Reporter
	Comment 1 • 9 years ago

Build ID               20150326010205
Gaia Revision          8dc256a2de273be3abfa2cb2103d872d677834f7
Gaia Date              2015-03-25 22:34:28
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/37d3dcbf23a9
Gecko Version          39.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20140925.191337
Firmware Date          Thu Sep 25 19:13:47 EDT 2014
Bootloader             L1TC10011800

	Johan Lorenzo [:jlorenzo]
	Comment 2 • 9 years ago

Repro'd on:

Build ID               20150326010205
Gaia Revision          8dc256a2de273be3abfa2cb2103d872d677834f7
Gaia Date              2015-03-25 22:34:28
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/37d3dcbf23a9
Gecko Version          39.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150326.043801
Firmware Date          Thu Mar 26 04:38:10 EDT 2015
Bootloader             L1TC000118D0

QA wanted for a branch check.

	Pi Wei Cheng [:piwei] (inactive)
	Comment 3 • 9 years ago

This issue does NOT occur on Flame 2.2. Answered calls don't appear under Missed Call log.

Device: Flame 2.2 (full flash 319MB KK)
BuildID: 20150326002504
Gaia: e59ac067a1d22b7a72cbebc892ec652723f2a557
Gecko: 04b4b9d1faae
Gonk: ebad7da532429a6f5efadc00bf6ad8a41288a429
Version: 37.0 (2.2) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0

	KTucker [:KTucker][Inactive 3/4/2016]
	Comment 4 • 9 years ago

Nominating this 3.0? since this a regression and bad from an end user's perspective.

	Pi Wei Cheng [:piwei] (inactive)
	Comment 5 • 9 years ago

b2g-inbound regression window:

Last Working Environmental Variables:
Device: Flame
BuildID: 20150324184620
Gaia: 419b6873b4dc310e0099ba269dd5364985b76539
Gecko: 771b34ea0153
Version: 39.0a1 (3.0 Master)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0

First Broken Environmental Variables:
Device: Flame
BuildID: 20150324201411
Gaia: 04924cd17fa7d6881c73e7d9a415410b0252cb9f
Gecko: 7690d9e5ba8a
Version: 39.0a1 (3.0 Master)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0

Last Working Gaia & First Broken Gecko - issue DOES repro
Gaia: 419b6873b4dc310e0099ba269dd5364985b76539
Gecko: 7690d9e5ba8a

Last Working Gecko & First Broken Gaia - issue does NOT repro
Gaia: 04924cd17fa7d6881c73e7d9a415410b0252cb9f
Gecko: 771b34ea0153

Gecko pushlog:
http://hg.mozilla.org/integration/b2g-inbound/pushloghtml?fromchange=771b34ea0153&tochange=7690d9e5ba8a

Caused by patches for Bug 1137093.

	KTucker [:KTucker][Inactive 3/4/2016]
	Comment 6 • 9 years ago

Szu-Yu, can you take a look at the please? This could have been caused by the landings for bug 1137093.

	Szu-Yu Chen [:aknow] (inactive) Assignee
	Comment 8 • 9 years ago

We should update the state first. Then set the |started| based on that.

	Hsin-Yi Tsai (she/her) [:hsinyi]
	Comment 9 • 9 years ago

Comment on attachment 8584273 [details] [diff] [review]
#2 Fix defect of setting call started time

Review of attachment 8584273 [details] [diff] [review]:
-----------------------------------------------------------------

Aknow, can you please help add a test case for this? We probably need to set navigator.mozSetMessageHandler for the system message "telephony-call-ended" and check the duration.

	Wesley Huang [:wesley_huang] (EPM) (NI me)
	Comment 10 • 9 years ago

triage: major regression, breaking basic function.

	Peter Bylenga [:PBylenga]
	Comment 11 • 9 years ago

This also fails the smoketests, upgrading severity.

	Szu-Yu Chen [:aknow] (inactive) Assignee
	Comment 12 • 9 years ago

try
https://treeherder.mozilla.org/#/jobs?repo=try&revision=02bd65248727

	Szu-Yu Chen [:aknow] (inactive) Assignee
	Comment 13 • 9 years ago

https://hg.mozilla.org/integration/b2g-inbound/rev/17b24c5f7773

	Szu-Yu Chen [:aknow] (inactive) Assignee
	Comment 14 • 9 years ago

I've done the local test for the patch so land it first to avoid blocking the smoke test.

Now, I am working on a new marionette test for the issue. Please let the bug open and I'll land the test case later.

	Phil Ringnalda (:philor)
	Comment 15 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/17b24c5f7773

	Martin Shuman [:Marty]
	Comment 16 • 9 years ago

This issue is verified fixed in the latest Nightly Flame 3.0
Answered calls appear properly in the call log.

Environmental Variables:
Device: Flame 3.0 (319MB)(Full Flash)
Build ID: 20150330010204
Gaia: be25b16efa19bab8d54be08f8fe45dcc93bf93d0
Gecko: dfe60814eda7
Gonk: b83fc73de7b64594cd74b33e498bf08332b5d87b
Version: 39.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0

	Hsin-Yi Tsai (she/her) [:hsinyi]
	Comment 18 • 9 years ago

Comment on attachment 8585982 [details] [diff] [review]
Test system message telephony-call-ended

Review of attachment 8585982 [details] [diff] [review]:
-----------------------------------------------------------------

It's really good to have this test, thank you.

	Szu-Yu Chen [:aknow] (inactive) Assignee
	Comment 19 • 9 years ago

try
https://treeherder.mozilla.org/#/jobs?repo=try&revision=d10b496efd02

	Szu-Yu Chen [:aknow] (inactive) Assignee
	Comment 20 • 9 years ago

https://hg.mozilla.org/integration/b2g-inbound/rev/0a5f6d6035de

	Ryan VanderMeulen [:RyanVM]
	Comment 21 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/0a5f6d6035de