510325 - update fennecmark as old version is broken with tile merge

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

http://people.mozilla.com/~tglek/fennec/fennecmark.xpi

Comment 2

[Aki Sasaki (not active)]

This is my attempt at merging, which was pretty much me dumping Taras' js into the tree.  At second glance it looks like Joel made changes which I overwrote, but I'm not sure what changes we need to keep.

http://hg.mozilla.org/users/asasaki_mozilla.com/talos-maemo/rev/1008d7631c94

Currently Fennecmark is timing out afaict.

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

I am at a loss here.  I have over 6 hours in trying to get the new fennecmark working.  While looking in the js error console, I see:

Error: invalid label
Source File: chrome://fennecbench/content/pageload.js Line: 35
browser.loadURI("http://www.yahoo.com", null, null, false);

Error: LagDuringLoad is not defined
Source File: chrome://fennecbench/content/overlay.js Line: 3



Then while clicking on the overlay image, I see this:
Error: BenchFE is undefined
Source File: chrome://browser/content/browser.xul  Line: 1


Not sure how to work around this.  If somebody can give me some pointers, I would appreciate it.

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: updated tpan

this diff file is a patch for:
http://hg.mozilla.org/users/tglek_mozilla.com/fennecmark

This patch adds in the extra talos stuff and when called via the event listener it sets the talos flag in BenchFE and report so the addtional talos magic happens (including loading the localhost page).

I have testing this as a .jar file with talos and standalone for linux desktop and it works as expected.  What is missing from this patch to run on talos is the fennecmark.html file which is a small file which triggers the event to start talos.  There is a Makefile (usage: make all) which zip's up the bits and creates a fennecbench.jar file.

taras, if you could review this patch and integrate it into your repository, we can then start using this on the maemo talos again.

Comment 5

[(dormant account)]

Comment on attachment 408107 [details] [diff] [review]
updated tpan

not sure why you need parseInt, but that wont hurt.

Comment 6

[Joel Maher ( :jmaher ) (UTC -8)]

Taras, could you apply the patch to your repo:
http://hg.mozilla.org/users/tglek_mozilla.com/fennecmark

Comment 7

[(dormant account)]

http://hg.mozilla.org/users/tglek_mozilla.com/fennecmark/rev/8cfdb627cca0

Comment 8

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: fennecmark driver for talos

Aki: please add fennecmark.html to the talos cvs repository.  This will drive fennecmark.  According to the mobile.config in cvs, it is referencing tpan/fennecmark.html for tpan and tzoom, so we can put it there and not have to edit anything else.

Comment 9

[Aki Sasaki (not active)]

Comment on attachment 408846 [details]
fennecmark driver for talos

this would go in tpan/fennecmark.html

Comment 10

[Aki Sasaki (not active)]

Attachment: mobile.config update to put fennecmark.html in talos/startup_test/

Aha, cvs diff -N includes the html file =)

Comment 11

[Aki Sasaki (not active)]

Comment on attachment 409213 [details] [diff] [review]
mobile.config update to put fennecmark.html in talos/startup_test/

Checking in mobile.config;
/cvsroot/mozilla/testing/performance/talos/mobile.config,v  <--  mobile.config
new revision: 1.2; previous revision: 1.1
done
RCS file: /cvsroot/mozilla/testing/performance/talos/startup_test/fennecmark.html,v
done
Checking in startup_test/fennecmark.html;
/cvsroot/mozilla/testing/performance/talos/startup_test/fennecmark.html,v  <--  fennecmark.html
initial revision: 1.1
done

Comment 12

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: makefile and manifest patch

Forgot to hg add Makefile and fennecbench.manifest.  Makefile will create the .jar file and fennecbench.manifest is needed in the chrome directory with the .jar file.

Comment 13

[Aki Sasaki (not active)]

Comment on attachment 410704 [details] [diff] [review]
makefile and manifest patch

I've pushed this; thanks Joel and Taras!