Closed
Bug 1195710
Opened 9 years ago
Closed 9 years ago
[MAP]Implement PushMessage function
Categories
(Firefox OS Graveyard :: Bluetooth, defect)
Tracking
(feature-b2g:2.2r+, firefox43 fixed, b2g-v2.2r fixed)
People
(Reporter: shawnjohnjr, Assigned: shawnjohnjr)
References
Details
Attachments
(2 files, 4 obsolete files)
Implement PushMessage function.
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → shuang
Assignee | ||
Updated•9 years ago
|
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8649267 -
Flags: review?(btian)
Assignee | ||
Comment 2•9 years ago
|
||
Comment on attachment 8649267 [details] [diff] [review]
bug1195710-mc.patch
I think I need to handle bMessage body first, cancel the review.
Attachment #8649267 -
Attachment is obsolete: true
Attachment #8649267 -
Flags: review?(btian)
Assignee | ||
Comment 3•9 years ago
|
||
Attachment #8657715 -
Flags: review?(btian)
Assignee | ||
Comment 4•9 years ago
|
||
Hi Ben,
I write bMessage parser in bug 1166647 which can take care of major work for parsing. For complete specification complaint, I prefer to keep some unused member variables/functions for native charset support.
Comment 5•9 years ago
|
||
Comment on attachment 8657715 [details] [diff] [review]
Bug 1195710 - [MAP]Implement PushMessage function
Review of attachment 8657715 [details] [diff] [review]:
-----------------------------------------------------------------
r=me with comment addressed. Please confirm whether to rebase on bug 1195685 that removes BT_APPEND_NAMED_VALUE macro.
::: dom/bluetooth/BluetoothCommon.h
@@ +251,5 @@
> /**
> + * When receiving a MAP request of 'push message' from a remote device,
> + * we'll dispatch an event.
> + */
> +#define MAP_PUSH_MESSAGE_REQ_ID "mappushmessagereq"
Group MAP related ID as PBAP [1].
[1] https://dxr.mozilla.org/mozilla-central/source/dom/bluetooth/common/BluetoothCommon.h#198
Attachment #8657715 -
Flags: review?(btian) → review+
Assignee | ||
Updated•9 years ago
|
Attachment #8657715 -
Attachment is obsolete: true
Assignee | ||
Comment 6•9 years ago
|
||
Assignee | ||
Comment 8•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Attachment #8660414 -
Attachment description: Bug 1195710 - [MAP]Implement PushMessage function, r=btian → Bug 1195710 - [MAP]Implement PushMessage function, r=btian (v2.2r)
Assignee | ||
Updated•9 years ago
|
Attachment #8660413 -
Attachment description: Bug 1195710 - [MAP]Implement PushMessage function, r=btian → Bug 1195710 - [MAP]Implement PushMessage function, r=btian (m-c)
Assignee | ||
Comment 9•9 years ago
|
||
Comment 10•9 years ago
|
||
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox43:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → FxOS-S7 (18Sep)
Assignee | ||
Updated•9 years ago
|
status-b2g-v2.2r:
--- → affected
Assignee | ||
Comment 11•9 years ago
|
||
Wesley,
Could you please set feature v2.2r+?
Flags: needinfo?(whuang)
Updated•9 years ago
|
feature-b2g: --- → 2.2r+
Flags: needinfo?(whuang)
Comment 12•9 years ago
|
||
Hi, this failed to apply:
dding 1195710 to series file
renamed 1195710 -> bug1195710-v22r.patch
applying bug1195710-v22r.patch
patching file dom/bluetooth/BluetoothCommon.h
Hunk #1 FAILED at 118
1 out of 1 hunks FAILED -- saving rejects to file dom/bluetooth/BluetoothCommon.h.rej
patching file dom/bluetooth/bluedroid/BluetoothMapSmsManager.cpp
Hunk #1 FAILED at 326
Hunk #2 FAILED at 817
Hunk #3 FAILED at 982
3 out of 3 hunks FAILED -- saving rejects to file dom/bluetooth/bluedroid/BluetoothMapSmsManager.cpp.rej
patching file dom/bluetooth/bluedroid/BluetoothMapSmsManager.h
Hunk #1 FAILED at 93
1 out of 1 hunks FAILED -- saving rejects to file dom/bluetooth/bluedroid/BluetoothMapSmsManager.h.rej
patch failed, unable to continue (try -v)
patch failed, rejects left in working directory
errors during apply, please fix and refresh bug1195710-v22r.patch
could you take a look, thanks!
Flags: needinfo?(shuang)
Assignee | ||
Comment 13•9 years ago
|
||
Hi
It looks like the current v2.2r doesn't include patch sets:
1. bug1166679-v22r.patch
2. bug1186836-v22r.patch
bug1195710-v22r.patch depends on those two patches.
Flags: needinfo?(shuang)
Assignee | ||
Updated•9 years ago
|
Flags: needinfo?(cbook)
Comment 14•9 years ago
|
||
Comment 15•9 years ago
|
||
sorry i had to back this out in https://hg.mozilla.org/releases/mozilla-b2g37_v2_2r/rev/67d81601af97
seems one change here was causing a bustage like https://treeherder.mozilla.org/logviewer.html#?job_id=6599&repo=mozilla-b2g37_v2_2r
Status: RESOLVED → REOPENED
Flags: needinfo?(cbook) → needinfo?(shuang)
Resolution: FIXED → ---
Assignee | ||
Updated•9 years ago
|
Attachment #8660414 -
Attachment is obsolete: true
Flags: needinfo?(shuang)
Assignee | ||
Comment 16•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Attachment #8662131 -
Attachment filename: bug1195710-v22r.patch → bug1195710-v22r.patch (v2.2r)
Assignee | ||
Updated•9 years ago
|
Attachment #8662131 -
Attachment description: Bug 1195710 - [MAP]Implement PushMessage function, r=btian → Bug 1195710 - [MAP]Implement PushMessage function, r=btian (v2.2r)
Attachment #8662131 -
Attachment filename: bug1195710-v22r.patch (v2.2r) → bug1195710-v22r.patch
Assignee | ||
Updated•9 years ago
|
Attachment #8662131 -
Attachment is obsolete: true
Assignee | ||
Comment 17•9 years ago
|
||
Rebase it again.
Assignee | ||
Comment 18•9 years ago
|
||
(In reply to Carsten Book [:Tomcat] from comment #15)
> sorry i had to back this out in
> https://hg.mozilla.org/releases/mozilla-b2g37_v2_2r/rev/67d81601af97
>
> seems one change here was causing a bustage like
> https://treeherder.mozilla.org/logviewer.html#?job_id=6599&repo=mozilla-
> b2g37_v2_2r
I found the bustage caused by this patch. I've rebase it again and it works locally. Sorry to increase your work load.
Flags: needinfo?(cbook)
Comment 19•9 years ago
|
||
Comment 20•9 years ago
|
||
(In reply to Shawn Huang [:shawnjohnjr] from comment #18)
> (In reply to Carsten Book [:Tomcat] from comment #15)
> > sorry i had to back this out in
> > https://hg.mozilla.org/releases/mozilla-b2g37_v2_2r/rev/67d81601af97
> >
> > seems one change here was causing a bustage like
> > https://treeherder.mozilla.org/logviewer.html#?job_id=6599&repo=mozilla-
> > b2g37_v2_2r
>
> I found the bustage caused by this patch. I've rebase it again and it works
> locally. Sorry to increase your work load.
no worries and absolutely no problem :)
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Flags: needinfo?(cbook)
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•