Closed
Bug 1004146
Opened 11 years ago
Closed 11 years ago
new Tarako Marketplace
Categories
(Firefox OS Graveyard :: Gaia, defect, P1)
Firefox OS Graveyard
Gaia
Tracking
(blocking-b2g:1.3T+, b2g-v1.3 affected, b2g-v1.3T fixed)
RESOLVED
FIXED
blocking-b2g | 1.3T+ |
People
(Reporter: clouserw, Assigned: clouserw)
References
Details
(Whiteboard: [tarako_only])
Attachments
(1 file, 2 obsolete files)
(deleted),
text/x-github-pull-request
|
Details |
We have a new marketplace for Tarako. Marking this 1.3T+
PR is at https://github.com/mozilla-b2g/gaia/pull/18845
Attachment #8415551 -
Flags: review?(felash)
Comment 1•11 years ago
|
||
Comment on attachment 8415551 [details]
update tarako marketplace; 2014-04-30
Stealing review.
Attachment #8415551 -
Flags: review?(felash) → review+
Comment 2•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
status-b2g-v1.3:
--- → fixed
Resolution: --- → FIXED
Whiteboard: [tarako_only]
Comment 3•11 years ago
|
||
Reverted due to breaking the tree and not having a proper commit message: https://github.com/mozilla-b2g/gaia/commit/ccb0ac659c209cd84e2e6be9887cbf22faa18334
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Updated•11 years ago
|
Assignee: nobody → clouserw
Updated•11 years ago
|
Assignee | ||
Comment 4•11 years ago
|
||
https://github.com/mozilla-b2g/gaia/pull/18867 has a commit message with this bug number in it
Assignee | ||
Comment 5•11 years ago
|
||
1403 kgrandon │ clouserw: if you want to needinfo? me I can ask some folks on the a-team to lend us a hand.
Flags: needinfo?(kgrandon)
Comment 6•11 years ago
|
||
Hey Zac - any chance for a quick look at the pull request in comment 4 here? We're updating Marketplace for 1.3t to remove the iframe, and when we do - the test seems to fail.
Flags: needinfo?(kgrandon) → needinfo?(zcampbell)
Comment 7•11 years ago
|
||
(In reply to Kevin Grandon :kgrandon from comment #6)
> Hey Zac - any chance for a quick look at the pull request in comment 4 here?
> We're updating Marketplace for 1.3t to remove the iframe, and when we do -
> the test seems to fail.
Afraid the failure is legitimate - see the report attachment.
(Apologies this report has been broken for some time on Travis but is easy to generate locally)
Flags: needinfo?(zcampbell)
Comment 8•11 years ago
|
||
Pardon I forgot to say, the test modification is fine.
I think the test is failing because Marionette is trying to switch to the frame but it's being destroyed before the action can be completed.
Thus if the marketplace does load properly the test should be fine.
Assignee | ||
Comment 9•11 years ago
|
||
It's easy to generate the Marketplace not loading locally? What are you seeing? It looks like you wrote the original tests - I need help repairing them for this patch. I'm not a gaia dev but this is a 1.3T+ bug. I've looked for you on IRC for a couple of days but haven't seen you - I'm clouserw on there, let's chat.
Comment 10•11 years ago
|
||
Thing is it's not a problem with the test.
Travis is using this binary:
http://ftp.mozilla.org/pub/mozilla.org/b2g/nightly/latest-mozilla-b2g28_v1_3/b2g-28.0.multi.linux-x86_64.tar.bz2
Then it's using a gaia profile generated with "DESKTOP_SHIMS=1 make"
If you dl/unpack that binary, then build a Gaia profile as above.
Then start then using "b2g-bin -profile /path/to/profile" , complete the FTU then try to load marketplace you'll get the same error.
In the console it reads:
JavaScript error: about:neterror?e=fileNotFound&u=app%3A//marketplace.firefox.com/index.html&c=UTF-8&f=app&m=https%3A//marketplace.firefox.com/marketplace-package.webapp&d=Firefox%20can%27t%20find%20the%20file%20at%20app%3A//marketplace.firefox.com/index.html., line 560: NS_ERROR_FILE_NOT_FOUND: Component returned failure code: 0x80520012 (NS_ERROR_FILE_NOT_FOUND) [nsIDOMLocation.reload]
Updated•11 years ago
|
Flags: needinfo?(kgrandon)
Comment 11•11 years ago
|
||
I'm having trouble with my Tarako and can't spend too much more time on this. We've verified that the marketplace works on a device, so let's disable the test and ship this. I've filed bug 1005241 to re-enable the test.
Flags: needinfo?(kgrandon)
Comment 12•11 years ago
|
||
Attachment #8415551 -
Attachment is obsolete: true
Attachment #8416439 -
Attachment is obsolete: true
Comment 13•11 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
status-b2g-v1.3T:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•