If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Create MMS workloads for performance testing

RESOLVED FIXED

Status

Firefox OS
Gaia::SMS
P2
normal
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: julienw, Assigned: jhylands)

Tracking

({perf})

unspecified

Firefox Tracking Flags

(b2g18+)

Details

(Whiteboard: c= p=5 ,)

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

4 years ago
For testing the performance of our MMS work, we now need some comprehensive MMS workloads.

* at least one big thread with MMS
* threads with mixed MMS and SMS
* MMS with different types of content: SMIL or not SMIL, text, images, video, sound
* some MMS with only one content, some MMS with several media

John, if you have some questions, you could probably ask to me or Corey (:gnarf) depending on the time of your day ;)
(Assignee)

Updated

4 years ago
Whiteboard: c=performance
(Assignee)

Updated

4 years ago
Whiteboard: c=performance → c=performance p=3
(Assignee)

Updated

4 years ago
Status: NEW → ASSIGNED
Please reference | MmsMessage::Create| in http://mxr.mozilla.org/mozilla-central/source/dom/mobilemessage/src/MmsMessage.cpp

Looks like you miss expiryDate. see line 258 in MmsMessage.cpp.
(Assignee)

Comment 2

4 years ago
So after digging further, it appears to be a problem with the first (and only) attachment. The content field is expecting a Blob object, and I am passing in a string.

However, I changed my code from this:

  msg.parts[1]["content"] = textContent;

to this:

  msg.parts[1]["content"] = new Blob([textContent], {type: 'text/plain'});

and now it is complaining "build/fakeSms.js:119: ReferenceError: Blob is not defined"

Keep in mind this is not running in Gaia on the phone, it is running inside a XUL-RUNNER script running on the desktop, called from the Makefile.
(Assignee)

Comment 3

4 years ago
Created attachment 759927 [details] [diff] [review]
Partial patch (not working) for Blob support in XPCShell scripts.

This needs to be applied against a b2g-desktop m-c repo. It currently does not build.
(Assignee)

Updated

4 years ago
Flags: needinfo?(josh)

Comment 4

4 years ago
Some info about the build errors would be easier for me to diagnose. However:

<jdm> jhylands: it's possible that NS_NewDOMMultipartFile and NS_NewDOMBlob are being defined inside a mozilla namespace.

If that's the case, they should just be moved to the bottom of the file, outside of any namespace blocks.
Flags: needinfo?(josh)

Updated

4 years ago
Keywords: perf
Whiteboard: c=performance p=3 → c= p=3 ,
(Assignee)

Comment 5

4 years ago
Created attachment 763736 [details]
Pull request - Added MMS reference workloads
Attachment #759927 - Attachment is obsolete: true
Attachment #763736 - Flags: review?(felash)
(Reporter)

Comment 6

4 years ago
Comment on attachment 763736 [details]
Pull request - Added MMS reference workloads

more comments on github
(Assignee)

Updated

4 years ago
Whiteboard: c= p=3 , → c= p=5 ,

Updated

4 years ago
tracking-b2g18: --- → +
Priority: -- → P2
(Reporter)

Comment 7

4 years ago
Comment on attachment 763736 [details]
Pull request - Added MMS reference workloads

working db in github now, more requests so that we can have a good workload to show off bugs.
(Reporter)

Comment 8

4 years ago
Comment on attachment 763736 [details]
Pull request - Added MMS reference workloads

r=me

will file more bugs for other less important features

thanks !
Attachment #763736 - Flags: review?(felash) → review+
(Reporter)

Comment 9

4 years ago
master: 11d5c75a900689f1779187d08212b923b9b15021
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.