Development :  K-Meleon Forum
K-Meleon development related discussions. 
Pages: Previous12
Current Page: 2 of 2
Re: builing v7 on moz 1.2.1
Posted by: MonkeeSage
Date: February 28, 2003 05:54AM

asmpgmr:

I tested all 4, and they did exactly as you described. Behavior as expected. And all of them now save to the same dir. smiling smiley

Great job, again!


Shelumi`El
Jordan

S.D.G

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: Andrew
Date: February 28, 2003 06:07AM

asmpgmr,

Please send it to the developer's list. We can test it and then create some diffs so the changes can be checked in.

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: asmpgmr
Date: February 28, 2003 08:21AM

Andrew,

I once tried to send something to the developer list and it never appeared there
presumably because you have to be subscribed for stuff to be accepted. I would
rather not get email every time someone sends something to the list when I can
instead look at the mailing list archive. Also the list is sometimes spammed which
is something I definitely do not want. Since I keep my email address fairly private,
I currently get ZERO spam and want to keep it that way.

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: Andrew
Date: February 28, 2003 11:32AM

Maybe MonkeeSage will post those to the list then. I'm not trying to force people to use the list but I know most of the developers don't follow all of the forum traffic and I can't say I see everything myself. I'll try and keep up with these changes. If you file them to the individual bugs, those will be seen by the developers.

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: Andrew
Date: February 28, 2003 12:29PM

asmpgmr,

Go ahead and e-mail me directly. I'll double-check those against the code and then pass along diffs so that we can check those into CVS. I'll probably go ahead and pass those along to Mozilla too for their MFCEmbed app.

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: Andrew
Date: February 28, 2003 12:50PM

If you are building on 1.3, you can also update your code for this:

http://bugzilla.mozilla.org/show_bug.cgi?id=194954

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: Andrew
Date: February 28, 2003 01:24PM
Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: ra
Date: February 28, 2003 08:39PM

Andrew, please notice the crash bug I reported in the Mozi-Meleon 1.3b-thread with the changes to save "web page, complete". Other than that the save-changes are really nice (and worth the crash ;-) ).

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: asmpgmr
Date: February 28, 2003 10:01PM

Andrew,

I sent you the updates.

ra,

You're saying the crash only occurs when using web page complete to save LOCAL
files ?? Does this happen with the released K-Meleon ? I would like to know if it
happens with the new code using Mozilla 1.2.1, I don't trust Mozilla 1.3b.

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: ra
Date: February 28, 2003 11:10PM

asmpgmr,

yes, *only* *local* pages, only with "web page, complete" and only with
MonkeeSages executable (installed with moz 1.3b exactly like described
in install.txt [1]).

The only thing that is saved is an empty directory _files. Repro-steps can
be found in the other thread. BTW: KM crashes only if I start (or go on)
browsing *internet* sites afterwards, if I browse local files, KM doesn't
crash. Can anyone repro the crash? MonkeeSage, does your KM crash?

I don't have a 1.2.1-based K-M here, but the service pack release (using
moz 1.2b-code) is working fine. Page is saved, images are there, no crash
afterwards, everything fine.

BTW: OS is Win XP Pro SP1.

[1] Well, okay, "nsIQTScriptablePlugin.xpt" is missing here, but that
shouldn't make a difference, should it?

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: asmpgmr
Date: February 28, 2003 11:33PM

ra,

Maybe MonkeeSage will make a Mozilla 1.2.1 build with the updated code, it could
be something different about the Mozilla 1.3 API that's causing the problem.
Out of curiosity, what made you try save web page complete on local files ?

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: MonkeeSage
Date: March 01, 2003 03:24AM

I'm building 1.2.1 at this moment. There is also a cookie issue with my build, and I'm curious if it is a 1.3 bug (I am betting it is). No cookies.txt is written with the 1.3 build, it reads them fine, but it is not writting them...I am thinking it is something in the 1.3 cookies.dll / .xpt in components.

As for the save complete, it works fine for me on local files with 1.3 and no crashes, so I'm not sure what the deal is. I thought I had changed the baracketing on this section slightly:

if(!bDocument) {
CProgressDialog *progress = new CProgressDialog(false);
progress->InitPersist(aURI, file, persist, TRUE);
persist->SaveURI(aURI, nsnull, nsnull, nsnull, nsnull, file);
}
else
if(bSaveAll)
persist->SaveDocument(nsnull, file, dataPath, nsnull, 0, 0);
else
persist->SaveURI(nsnull, nsnull, nsnull, nsnull, nsnull, file);
}

But obviously not, because that is the exact cose asmpgmr listed above.

I'll post the 1.2.1 binary when I'm done compiling (might be about a hr or so judging from the 1.3 compile time).


Shelumi`El
Jordan

S.D.G

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: asmpgmr
Date: March 01, 2003 04:19AM

MonkeeSage,

Careful, that final } matches if(cf.DoModal() == IDOK) { which is far above.
If there's a problem then perhaps it in the 1.3 SaveDocument function call
considering SaveURI is different between 1.2 and 1.3.

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: MonkeeSage
Date: March 01, 2003 05:43AM

asmpgmr:

If you would, so I don't have to go back through the forums and get the pieces, will you please send the document with your code revisions to: my email ?

Thanks!


Shelumi`El
Jordan

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: MonkeeSage
Date: March 01, 2003 05:44AM
Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: MonkeeSage
Date: March 01, 2003 07:54AM

Oh yeah...I forgot to mention this before...

A big fat BTW needs to be attached to pulling K-M off CVS using cygwin

It sets the .dsw and .dsp files to UNIX file-type, instead of DOS, so you have to set them back manually (e.g., with UltraEdit32), before VC++ will recognize them, otherwise it shows no projects in the main .dsw!! smiling smiley


Shelumi`El
Jordan

S.D.G

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: MonkeeSage
Date: March 01, 2003 11:13AM

m4ster:

Here is a MOZCONFIG file to optimize building Mozilla:

[---snip---]

# sh
# Build configuration script
#
# See http://www.mozilla.org/build/unix.html for build instructions.
#

# Options for 'configure' (same as command-line options).
ac_add_options --disable-mailnews
ac_add_options --disable-ldap
# ac_add_options --enable-ctl
ac_add_options --disable-composer
ac_add_options --enable-activex-scripting
ac_add_options --disable-tests
ac_add_options --enable-xpctools
ac_add_options --disable-debug
ac_add_options --enable-optimize=-O3
ac_add_options --enable-optimize=-fsave-memoized
ac_add_options --enable-optimize=-fexpensive-optimizations

[---snip---]

Name the file something like "mozconfig," then if you are using cygwin & bash, run the command:

export MOZCONFIG=C:\fully\qualified\path\to\mozconfig

Or from cmd.exe, run the command:

set MOZCONFIG=C:\fully\qualified\path\to\mozconfig

Make sure to export / set this environment variable prior to building (like in your env.sh script), or Mozilla won't know about your configuration and will build using the default. smiling smiley


Shelumi`El
Jordan

S.D.G

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: MonkeeSage
Date: March 01, 2003 11:34AM

Actually, use this one...-fexpensive-optimizations is only for MiNGW it seems:

[---sip---]

# sh
# Build configuration script
#
# See http://www.mozilla.org/build/unix.html for build instructions.
#

# Options for 'configure' (same as command-line options).
ac_add_options --disable-mailnews
ac_add_options --disable-ldap
ac_add_options --disable-composer
ac_add_options --disable-installer
ac_add_options --enable-activex-scripting
ac_add_options --disable-tests
ac_add_options --enable-xpctools
ac_add_options --disable-debug
ac_add_options --enable-optimize=-O3
ac_add_options --enable-optimize=-fsave-memoized

[---snip---]


Shelumi`El
Jordan

S.D.G

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: Andrew
Date: March 01, 2003 12:03PM

asmpgmr and others,

Any ideas how to handle a reload in an individual frame? I culled through the MFCEmbed code as well as the Mozilla code and I didn't see anything for us to hook into that would accomplish that. In Mozilla, it's done with Javascript which isn't really what I was looking for.

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: MonkeeSage
Date: March 01, 2003 12:56PM

Andrew:

Check out:

http://www.mozilla.org/docs/dom/mozilla/classes/htmlframe.html

These two looked like they might be helpful:

content/html/content/src/nsHTMLFrameElement.cpp
dom/public/idl/html/nsIDOMHTMLFrameElement.idl

Mabye to "refresh" a frame, you can grabe the frameURI, and add a new frame exactly where the old frame was in the layout, and simply pass the original frameURI as the URI to the new frame?


Shelumi`El
Jordan

S.D.G

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: asmpgmr
Date: March 01, 2003 10:13PM

Andrew,

I did look in Mozilla (http://lxr.mozilla.org/seamonkey/) but couldn't find the function
used to reload a frame, I only saw the Javascript stuff.

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: Andrew
Date: March 01, 2003 10:58PM

asmpgmr,

OK, well at least I know I didn't miss anything.
smiling smiley

I'll have to see if I can get the Mozilla people to look into that.

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: ra
Date: March 03, 2003 03:27AM

asmpgmr (02-28-03 11:33),

> Maybe MonkeeSage will make a Mozilla 1.2.1 build with the updated code,

Using the version from this afternoon.

> it could be something different about the Mozilla 1.3 API that's causing the
> problem.

Yeah, with 1.2.1 it works like a charm. :-)


> Out of curiosity, what made you try save web page complete on local files ?

I'm always trying to save the same page with *every* km build (I'm using km since
christophe released .1 back then), because there were *often* bugs in the save as-
function... It's quite a simple page, that lists a lot of searchengines with their
logos, that I linked from my (other) local startpage.

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: ra
Date: March 04, 2003 12:36AM

I wrote:

> Yeah, with 1.2.1 it works like a charm. :-)

Well, yeah, I can save *one* page and that's it. Then I have to completely close km,
open it again and can save another page and so on. So the fix / workaround (dialog-
box...) for the other bug is causing trouble as noted in another thread before.

That's "Version 0.7.1 Build 1 Compiled Tue Feb 27 09:02:53 2003" on Moz 1.2.1.

Options: ReplyQuote
Re: builing v7 on moz 1.2.1
Posted by: ra
Date: March 04, 2003 03:06AM

Me again...

Ah, well, just got the latest version and with that saving pages is broken again...
Guess I'll have to wait for MonkeeSage to make a new one...


Don't know if it makes much sense to report that now, but anyhow, there was
another bug with the save as-function in the older version:

The file extension won't be added if you choose to save a page that has a different
extension than your chosen save as-option.

Sounds difficult? ;-) Example: Save a php-page, let's call it "filename.php" as
"Web Page, HTML only (*.HTM;*.HTML)". Then you will get "filename.php"
instead of (what one would be exptecting) filename.htm or filename.php.htm.

Another one: If I save home.htm as "Text File (*.TXT)" the saved file will be
"home.htm".

It works when the shown extension is manually removed or changed to an
*unknown* file type.


And the cookie confirmation dialog is broken, too (checkbox doesn't work)...

So, I'm back to the 1.2b-code with KM.7SP1 for now. :-)

Options: ReplyQuote
Pages: Previous12
Current Page: 2 of 2


K-Meleon forum is powered by Phorum.