1
edit
Changes
→Firefox Build
='''FIREFOXTasks for Firefox'''===My works on firefox stuff=====Firefox Build===
For building firefox I followed the instructions on [http://annasob.wordpress.com/category/build/ annasob's blog]on how to build firefox on different platforms, also looked at build instructions and prerequisites @[https://developer.mozilla.org/En/Developer_Guide/Build_Instructions Mozilla Developing Center(MDC)].
I didn't figure out why it was so slow though on both Vista and Win 7.
First, I tried to build firefox on Vista:
: Intel(R) Processor D CPU 2.17 GHz : - 2GB DDR2 memory : - Windows Vista 32bit: - Visual Studio 2010
and this was my .mozconfig:
:<code>. $topsrcdir/browser/config/mozconfig</code>
:<code>mk_add_options MOZ_MAKE_FLAGS="-j2"</code>
:<code>mk_add_options MOZ_CO_PROJECT=browser</code>
:<code>ac_add_options --enable-application=browser</code>
:<code>ac_add_options --disable-optimize</code>
:<code>ac_add_options --enable-debug </code>
Then I started to build it on Windows 7:
: Intel(R) Processor D CPU 3.20 GHz : - 4GB DDR2 memory : - Windows 7 64bit: - Visual Studio 2008
I tried to build based on the same .mozconfig file, but the first error I got was related to SDK. I needed to upgrade SDK version based on these [https://developer.mozilla.org/En/Windows_SDK_versions recommendations] to SDK 7, and the .Net Framework to 4. Also I needed to change the SDK version in the SDK configuration tool to 7.1.
At the end, the build was successfully done after 2 hours on Win 7 and more than 4 hours on Vista!
[[File:Minefield_finalminefield_final.jpg|thumb|none|300px100px]]------------------------------------------
:Source Control: mercurial
:Blog: @ [http://Minooz.wordpress.com wordpress]
* problem'''problem1'''
: Open a new tab right beside the selected tab.
: Target file to be changed @[http://mxr.mozilla.org/mozilla-central/source/browser/base/content/tabbrowser.xml mozilla-central]
* Patch
: [[User:Minooz/OSD600/FF/patchNewTab | patchMoz_newTab]]
* '''problem2'''
: Instead of using alt+tab+num to switch between open tabs, use ctr+tab+num.
: Target file to be changed @[http://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser-sets.inc mozilla-central]
* Patch
: [[User:Minooz/OSD600/FF/patchCtrlTab | patchMoz CtrlTab]]
------------------------------------------
:Source Control: mercurial
:BugTracker: bugzilla
:Blog: @ [http://Minooz.wordpress.com wordpress]
* '''Bugs'''
: Make is_space() in nsImageMap.cpp HTML5-compliant. Ticket @[https://bugzilla.mozilla.org/show_bug.cgi?id=565031 bugzilla #565031]
: Target file to be changed @ [http://mxr-test.konigsberg.mozilla.org/mozilla-central/source/layout/generic/nsImageMap.cpp#110 mozilla-central]
* Helpful tips
: * Patches [http: 1//mxr.mozilla.org/mozilla- patchMoz_Apr6 <central/source lang="java"> diff -r ec44e31cc46b /layout/generic/nsImageMaptest/test_bug448860.cpphtml?force=1 sample bug_test_1]-[http://mxr.mozilla.org/mozilla-- acentral/source/layout/generic/nsImageMaptest/test_bug290397.cpp Wed Mar 30 20html?force=1 sample_2]- [http:48:32 2011 //mxr.mozilla.org/mozilla-0400+++ bcentral/source/layout/generic/nsImageMaptest/test_bug416168.cpp Wed Apr 06 22:11:22 2011 -0400@@ -107,18 +107,21 @@ Area::~Area() delete [html sample_3] mCoords; } #include <stdlib.h> inline PRBool is_space(char c) {- return (c == * Patches for ' ' ||- c == nsImageMap.cpp'\f' ||+ return (c == ' '); +}+not_space(char c)+{+ return (c == '\f' || c == '\n' || c == '\r' |: [[User:Minooz/OSD600/FF/patch565031#Apr6 |patchMoz_Apr6]] c == '\t' || c == '\v'); } static void logMessage(nsIContent* aContent, const nsAString& aCoordsSpec,@@ -181,21 +184,33 @@ void Area:Still working...[[User:ParseCoords(const nsAString& while (*n_str != '\0') { PRBool has_comma; Minooz/OSD600/* * Skip to a separator *FF/ tptr = n_str;- while (!is_space(*tptr) && *tptr != ',' && *tptr != '\0')+ while (!is_space(*tptr) && *tptr != ',' && *tptr != ';' && *tptr != '\0') {- tptr++;+ if (not_space(*tptr))+ {+ break;+ }+ else + {+ tptr++;+ } } n_str = tptr;+ + if (*tptr == ';')+ {+ *tptr = ',';+ }patch565031#Apr16 | patchMoz_Apr16]] /* * If no more entries, break out hereTest Page */ if (*n_str == '\0') { break; }@@ -256,17 +271,17 @@ void Area: [[User:ParseCoords(const nsAString& { return; } Minooz/* * Second pass to copy integer values into list. *OSD600/ tptr = cp;- for (i=0; i<cnt; i++)+ for (i = 0; i < cnt; i++) { char *ptr; ptr = strchr(tptr, ','); if (ptr) { *ptr = '\0'; } <FF/source>test565031 | test_bug560531.html]]
------------------------------------------------