Crash when revCloseBrowser

[Mike Bonner]

Its not the issue, but wanted to point out that you can simplify your
browser closing handler slightly.

on closeBrowser
   put revBrowserInstances() into tInstances
   repeat for each item tInst in tInstances
      if tInstances is not empty then -- don't need this
         revBrowserClose tInst
      end if -- or this
   end repeat
end closeBrowser

If there isn't anything in tinstances it won't be repeating, so theres no
need to do the check for tInstances not empty.

If anything, you might do this instead: if tInst is an integer then
  revbrowserclose tInst
else
  answer information "something is wrong, tInst is " & tInst -- if tinst
isn't an integer, what is it?
end if

You also might wrap your revbrowserclose in a try/catch

On Mon, Jan 12, 2015 at 8:35 AM, Earthednet-wp <prothero at earthednet.org>
wrote:

> Dixie,
> That's what I see. It works fine on osx but crashes on windows
> standalones. Works ok in windows IDE.
> Bill
>
> William Prothero
> http://es.earthednet.org
>
> > On Jan 11, 2015, at 11:42 PM, John Dixon <dixonja at hotmail.co.uk> wrote:
> >
> >
> > Bill...
> >
> > I'm not too sure about your closeCard handler... I am not seeing
> crashes, but there again, I'm not running windows, but am not having any
> problems using OSX Yosemite, LC 7.0
> >
> > local browserID
> >
> > ......
> >
> > on closeCard
> >   /* destroy the browser instance when the card closes */
> >   if browserID is not empty then
> >      revBrowserClose browserID
> >      put empty into browserID
> >   end if
> > end closeCard
> >
> > Dixie
> >
> >> From: prothero at earthednet.org
> >> Subject: Crash when revCloseBrowser
> >> Date: Sun, 11 Jan 2015 22:47:30 -0800
> >> To: use-livecode at lists.runrev.com
> >>
> >> Folks:
> >> Is anybody using the browser window in Windows? I get a crash on
> revBrowserClose, in a standalone. It works in the IDE. The browser opens
> and loads the web page, a simple text page then crashes the app when I
> close it. I’m on Windows 7, 64 bit, Livecode 7.0.1, using Parallels 10 on
> Yosemite. The CEF version of the browser also crashes the app when I try to
> close it.
> >>
> >> This is the last issue before I have a first version of a standalone
> app that works on Mac and PC desktops. For now, unless somebody has a
> bright idea, I’ll have to launch the default browser on Windows systems to
> get what I want, which is lame.
> >>
> >> I’ll submit a bug report unless someone points out an error in my code.
> >>
> >> When I open the browser:
> >>   put the windowID of this stack into tWinID
> >>      put revBrowserOpen(tWinID,helpURL) into sBrowserID
> >>
> >> —When I close the browser
> >> on closeBrowser
> >>   put revBrowserInstances() into tInstances
> >>   repeat for each item tInst in tInstances
> >>      if tInstances is not empty then
> >>         revBrowserClose tInst
> >>      end if
> >>   end repeat
> >> end closeBrowser
> >>
> >> Regards,
> >> Bill
> >>
> >>
> >> William A. Prothero
> >> http://es.earthednet.org/
> >>
> >> _______________________________________________
> >> use-livecode mailing list
> >> use-livecode at lists.runrev.com
> >> Please visit this url to subscribe, unsubscribe and manage your
> subscription preferences:
> >> http://lists.runrev.com/mailman/listinfo/use-livecode
> >
> > _______________________________________________
> > use-livecode mailing list
> > use-livecode at lists.runrev.com
> > Please visit this url to subscribe, unsubscribe and manage your
> subscription preferences:
> > http://lists.runrev.com/mailman/listinfo/use-livecode
>
> _______________________________________________
> use-livecode mailing list
> use-livecode at lists.runrev.com
> Please visit this url to subscribe, unsubscribe and manage your
> subscription preferences:
> http://lists.runrev.com/mailman/listinfo/use-livecode
>