Closed
Bug 1199107
Opened 9 years ago
Closed 9 years ago
Access PBAP vCard input stream directly instead of passing it as input parameter
Categories
(Firefox OS Graveyard :: Bluetooth, defect)
Tracking
(feature-b2g:2.2r+, b2g-v2.2r fixed, b2g-master fixed)
RESOLVED
FIXED
feature-b2g | 2.2r+ |
People
(Reporter: ben.tian, Assigned: ben.tian)
References
Details
Attachments
(4 files, 4 obsolete files)
(deleted),
patch
|
shawnjohnjr
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
shawnjohnjr
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
Access member variable of vCard input stream directly instead of passing it as input parameter.
Assignee | ||
Comment 1•9 years ago
|
||
Assignee: nobody → btian
Attachment #8653313 -
Flags: review?(shuang)
Assignee | ||
Updated•9 years ago
|
Attachment #8653313 -
Attachment description: Patch 1 (v1): Access PBAP vCard input stream directly instead of passing it as input parameter → [2.2r] Patch 1 (v1): Access PBAP vCard input stream directly instead of passing it as input parameter
Assignee | ||
Comment 2•9 years ago
|
||
Attachment #8653314 -
Flags: review?(shuang)
Assignee | ||
Comment 3•9 years ago
|
||
change:
- rename |mRequirePhonebookSize| to |mPhonebookSizeRequired|
Attachment #8653314 -
Attachment is obsolete: true
Attachment #8653314 -
Flags: review?(shuang)
Assignee | ||
Updated•9 years ago
|
Attachment #8653323 -
Flags: review?(shuang)
Assignee | ||
Comment 4•9 years ago
|
||
changes:
- remove __FUNCTION__ from log since BT_LOGR already includes it
- revise |Disconnect| with guardian clause
Attachment #8653323 -
Attachment is obsolete: true
Attachment #8653323 -
Flags: review?(shuang)
Attachment #8653325 -
Flags: review?(shuang)
Attachment #8653313 -
Flags: review?(shuang) → review+
Attachment #8653325 -
Flags: review?(shuang) → review+
Assignee | ||
Comment 5•9 years ago
|
||
Mark as feature-b2g:2.2r+ since this bug relates to 2.2r required feature PBAP.
Also set checkin-needed to land the 2 patches into 2.2r branch.
feature-b2g: --- → 2.2r+
Keywords: checkin-needed
Comment 6•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-b2g37_v2_2r/rev/217e2a1460a1
https://hg.mozilla.org/releases/mozilla-b2g37_v2_2r/rev/03dd969850a0
Is master unaffected/wontfix?
status-b2g-v2.2r:
--- → fixed
status-b2g-master:
--- → ?
Flags: needinfo?(btian)
Keywords: checkin-needed
Target Milestone: --- → FxOS-S6 (04Sep)
Assignee | ||
Comment 7•9 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #6)
> https://hg.mozilla.org/releases/mozilla-b2g37_v2_2r/rev/217e2a1460a1
> https://hg.mozilla.org/releases/mozilla-b2g37_v2_2r/rev/03dd969850a0
>
> Is master unaffected/wontfix?
M-c patch requires to rebase due to huge difference between m-c and 2.2r. I'm focusing on 2.2r first and will land the fix to m-c later. Please leave this bug open to track m-c fix.
Flags: needinfo?(btian)
Updated•9 years ago
|
Target Milestone: FxOS-S6 (04Sep) → ---
Assignee | ||
Comment 8•9 years ago
|
||
Assignee | ||
Comment 9•9 years ago
|
||
Assignee | ||
Comment 10•9 years ago
|
||
Add missing log revision.
Attachment #8654708 -
Attachment is obsolete: true
Assignee | ||
Comment 11•9 years ago
|
||
Revise patch 2 for JB emulator bustage.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=aea9051d5fcb
Attachment #8654757 -
Attachment is obsolete: true
Assignee | ||
Comment 12•9 years ago
|
||
try run that passes: https://treeherder.mozilla.org/#/jobs?repo=try&revision=93e758e4aff9
Comment 13•9 years ago
|
||
Assignee | ||
Comment 14•9 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/c17ef06059ed
http://hg.mozilla.org/mozilla-central/rev/3deec398d887
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•