Difference between revisions of "User:Minooz/OSD600/FF"

From CDOT Wiki
Jump to: navigation, search
(Firefox Bug (WhiteSpaces))
(Firefox Build)
 
(3 intermediate revisions by the same user not shown)
Line 17: Line 17:
 
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.  
 
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!
 
At the end, the build was successfully done after 2 hours on Win 7 and more than 4 hours on Vista!
[[File:Minefield_final.jpg|thumb|none|100px]]
+
[[File:minefield_final.jpg|thumb|none|100px]]
 
------------------------------------------
 
------------------------------------------
  
Line 23: Line 23:
 
:Source Control: mercurial
 
:Source Control: mercurial
 
:Blog: @ [http://Minooz.wordpress.com wordpress]
 
:Blog: @ [http://Minooz.wordpress.com wordpress]
* problem1
+
* '''problem1'''
 
: Open a new tab right beside the selected tab.
 
: 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]
 
: Target file to be changed @[http://mxr.mozilla.org/mozilla-central/source/browser/base/content/tabbrowser.xml mozilla-central]
 
* Patch
 
* Patch
 
: [[User:Minooz/OSD600/FF/patchNewTab | patchMoz_newTab]]
 
: [[User:Minooz/OSD600/FF/patchNewTab | patchMoz_newTab]]
* problem2
+
* '''problem2'''
 
: Instead of using alt+tab+num to switch between open tabs, use ctr+tab+num.
 
: 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]
 
: Target file to be changed @[http://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser-sets.inc mozilla-central]
Line 39: Line 39:
 
:BugTracker: bugzilla
 
:BugTracker: bugzilla
 
:Blog: @ [http://Minooz.wordpress.com wordpress]
 
:Blog: @ [http://Minooz.wordpress.com wordpress]
* Bugs
+
* '''Bugs'''
 
: Make is_space() in nsImageMap.cpp HTML5-compliant. Ticket @[https://bugzilla.mozilla.org/show_bug.cgi?id=565031 bugzilla #565031]
 
: 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]
 
: Target file to be changed @ [http://mxr-test.konigsberg.mozilla.org/mozilla-central/source/layout/generic/nsImageMap.cpp#110 mozilla-central]
 
* Helpful tips
 
* Helpful tips
: [http://mxr.mozilla.org/mozilla-central/source/layout/generic/test/test_bug448860.html?force=1 sample bug_test_1]
+
: [http://mxr.mozilla.org/mozilla-central/source/layout/generic/test/test_bug448860.html?force=1 sample bug_test_1]- [http://mxr.mozilla.org/mozilla-central/source/layout/generic/test/test_bug290397.html?force=1 sample_2]- [http://mxr.mozilla.org/mozilla-central/source/layout/generic/test/test_bug416168.html sample_3]
: [http://mxr.mozilla.org/mozilla-central/source/layout/generic/test/test_bug290397.html?force=1 sample bug_test_2]
 
: [http://mxr.mozilla.org/mozilla-central/source/layout/generic/test/test_bug416168.html sample bug_test_3]
 
 
* Patches for ''nsImageMap.cpp''
 
* Patches for ''nsImageMap.cpp''
 
: [[User:Minooz/OSD600/FF/patch565031#Apr6 | patchMoz_Apr6]]
 
: [[User:Minooz/OSD600/FF/patch565031#Apr6 | patchMoz_Apr6]]

Latest revision as of 21:40, 2 June 2011

Tasks for Firefox

Firefox Build

For building firefox I followed the instructions on annasob's blogon how to build firefox on different platforms, also looked at build instructions and prerequisites @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:

. $topsrcdir/browser/config/mozconfig
mk_add_options MOZ_OBJDIR=@TOPSRCDIR@/objdir-ff-release
mk_add_options MOZ_MAKE_FLAGS="-j2"
mk_add_options MOZ_CO_PROJECT=browser
ac_add_options --enable-application=browser
ac_add_options --disable-optimize
ac_add_options --enable-debug

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 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!

Minefield final.jpg

Firefox practice (Tabs)

Source Control: mercurial
Blog: @ wordpress
  • problem1
Open a new tab right beside the selected tab.
Target file to be changed @mozilla-central
  • Patch
patchMoz_newTab
  • problem2
Instead of using alt+tab+num to switch between open tabs, use ctr+tab+num.
Target file to be changed @mozilla-central
  • Patch
patchMoz CtrlTab

Firefox Bug (WhiteSpaces)

Source Control: mercurial
BugTracker: bugzilla
Blog: @ wordpress
  • Bugs
Make is_space() in nsImageMap.cpp HTML5-compliant. Ticket @bugzilla #565031
Target file to be changed @ mozilla-central
  • Helpful tips
sample bug_test_1- sample_2- sample_3
  • Patches for nsImageMap.cpp
patchMoz_Apr6
Still working... patchMoz_Apr16
  • Test Page
test_bug560531.html