Update debian/changelog, bump version number

Merge commit 'diablo-package-3.3-2' into fremantle-package

Update debian/changelog, bump version number

Provide open_new_window and load_url methods with boolean fullscreen argument

MicroB implements an optional boolean argument to the open_new_window
and load_url methods in the com.nokia.osso_browser interface, which
allows the application to request that the new page be loaded in a
fullscreen window.  We need to provide this as well -- otherwise,
applications that use the argument will receive an error.

Provide versions of open_new_window and load_url with a fullscreen
argument, which is ignored for the moment.  It's not going to be
possible to implement the fullscreen behavior for all supported
browsers, but we should implement it for MicroB, at least.

Fixes the launching of the Flickr new account page from the Sharing
control panel in Fremantle:
https://garage.maemo.org/tracker/index.php?func=detail&aid=6444&group_id=1159&atid=4348
Thanks to Tom G for reporting.

Update debian/changelog, bump version number

Fremantle: Make DBusGProxy static in launch_microb_open_window()

Chances are good we'll need to use the D-Bus proxy again at some point,
so instead of getting rid of it when launch_microb_open_window()
finishes, save it for later use.

Has the side effect of fixing a leak when we open a bookmarks window.

Fremantle: Use open_new_window instead of load_url when passing URL to MicroB

With the changes to MicroB launching for Fremantle, there are now cases
where we are passing a D-Bus request to a MicroB process with windows
open, instead of just asking for a browser session to be started.  In
the latter case, load_url and open_new_window are equivalent, but in the
former, they are not -- load_url loads the URL in an existing browser
window.

Fully emulating MicroB's behavior here now requires keeping track of
whether the request we got was load_url or open_new_window and
requesting the same of MicroB, but for now, do what most users want and
what we do with other browsers -- always open a new window for the URL.

Update debian/changelog, bump version number

Merge commit 'diablo-package-3.3-1' into fremantle-package

Update debian/changelog, bump version number

Merge commit 'v3.3' into diablo-package

Remove webpages for release

Update README

Update webpage

Users looking for beta packages for Fremantle (as opposed to development
releases) should use extras-testing, not extras-devel.

Update debian/changelog, bump version number

Update postinst

We can now coexist with a running MicroB browser process, so don't try
to kill off browser in the postinst.  On the other hand, we now need
browser-switchboard running at all times in order for things to work
correctly (especially if there is a MicroB browser process running), so
start browser-switchboard as the user.  We assume the Maemo default
user, since we have no good way of checking this, and on a regular
device (e.g. not the SDK) it's always true.

Update debian/rules

We need to build the Fremantle-specific version of the command-line
utility, and install the new Xsession.post script.

Merge branch 'diablo-package' into fremantle-package

Update debian/changelog, bump version number

Merge commit 'v3.3b2' into diablo-package

Restore webpages

Remove webpages for release

Make sure we don't try to free a random element of cfg in set_config_value()

Update Changelog

Update README, document changes in 3.3 betas

Command-line utility: avoid double free()s when modifying config option

Add install target for Fremantle Xsession.post script to Makefile

Fremantle: Prestart MicroB if appropriate when reconfiguring browser-switchboard

If, in changing the browser-switchboard configuration, we go from a
configuration where MicroB is not left running to one where it is, we
should start MicroB in the background in order to make sure that the
MicroB browser window comes up quickly when requested.  Make a
best-effort attempt at this in both the command-line utility and the
GUI, refactoring to give a swb_reconfig() function that can be shared
between the two along the way.

Ideally, we'd also kill MicroB when making a config change in the other
direction, but we don't have an easy way of knowing whether MicroB is
actually in use and we don't want to kill MicroB if it's in use at the
time.

Add an Xsession.post script for Fremantle to launch browser-switchboard on session startup

Fremantle MicroB autostarts by default and listens on both the D-Bus
system and session buses, so applications may send requests on either
bus (even though those that send requests on the system bus should be
fixed).  We therefore need to duplicate this behavior, but we cannot
rely on D-Bus to launch us to handle a request coming in on the system
bus, since the system bus runs as the "messagebus" user, and we need to
run with the UID of the logged-in user.  A complete fix for Browser
Switchboard bug 5910 ("browser-switchboard should listen on D-Bus system
bus as well") on Fremantle therefore requires that browser-switchboard
be autostarted when the desktop environment comes up.

We arrange for this the same way browserd does on Fremantle -- with an
Xsession.post script that runs on X session startup.  Also, we prestart
MicroB if the current browser-switchboard configuration is to leave it
running in the background, to make opening the first MicroB window
faster.

Because we may start the MicroB browser process, this script needs to
be inserted after browserd's in the Xsession.post run order.

Fix /usr/bin/microb script to work with new Fremantle MicroB launching code

The new Fremantle MicroB launching code breaks the assumption in the
microb script that the existence of the MicroB browser process implies
that MicroB is handling com.nokia.osso_browser.  Revise the script to
check directly whether Browser Switchboard is handling that name, by
comparing its owner to that of org.maemo.garage.browser-switchboard
(using the org.freedesktop.DBus.GetNameOwner method).

Make sure to initialize the DBusError in microb_start_dbus_watch_remove()

Fremantle: Coexist with a running MicroB process; don't kill MicroB at end of session if MicroB is the default browser

With commit aeb836f5... ("Tweak detection of com.nokia.osso_browser
acquisition slightly"), it's almost possible to support opening a new
window in an already-running MicroB.  All we have to do is avoid
invoking a new browser process in that case, so do that here.

It also turns out to be possible to take back the com.nokia.osso_browser
D-Bus name from MicroB without killing it, which is interesting to us
because the need to kill MicroB is what necessitates the behavior
changes we force on MicroB.

On the other hand, if we choose to avoid killing MicroB, we lose the
ability to have MicroB take over handling osso_browser requests while a
MicroB window is open, and we waste memory for users who aren't using
MicroB very often.

Therefore, we take a compromise approach: if MicroB is set as the
default browser, we don't bother killing MicroB when the last browser
window closes (which simplifies the code considerably, since we no
longer need to monitor MicroB while it's running, and also means that
MicroB should behave as it does when browser-switchboard isn't running).
Otherwise, we act as before (watch the running MicroB and kill it when the
last browser window closes, with the behavior changes this requires).

For control freaks, there's also a new autostart_microb config option
which allows you to override this heuristic.  Setting autostart_microb
to 0 forces us to kill MicroB even when MicroB is the configured default
browser, and setting it to 1 forces us to leave MicroB running even when
MicroB isn't the default.

Refactor Fremantle launch_microb code

Split out code that will be shared with the new code to launch a window
in an already-existing MicroB process to separate functions, and move
the rest of the existing hairball into a new
launch_microb_fremantle_with_kill() to make it easier to plug the
forthcoming launch_microb_fremantle() in.

Tweak detection of com.nokia.osso_browser acquisition slightly

Instead of looking for an empty old owner string, look for a nonempty
new owner string -- this picks up cases where ownership changes hands as
well as cases where a previously unowned name is acquired.

Whitespace fixes

Command-line utility: check that browser is installed before reporting it as the default

When reporting the configured default browser (-b), check to see that
it's installed.  If the configured default browser isn't installed,
report the default default browser instead, since that's what
browser-switchboard will actually use.

Config UI: Only offer the user browsers that are installed

Check to see which of the known browsers is installed, and only offer
the user choices for the ones that are.  If the currently configured
default browser is not installed (e.g. if the user uninstalled the
browser without changing the default), use the default default browser
like browser-switchboard does.

This fixes Browser Switchboard bug 5469:
https://garage.maemo.org/tracker/index.php?func=detail&aid=5469&group_id=1159&atid=4348

Fremantle: change "Control Panel" to "Settings menu" in package description

Merge branch 'diablo-package' into test-branch

Update package description

Emanuele Cassioli pointed out that the package description doesn't
mention that we ship a control panel applet to configure the default
browser, meaning that users who never bother to look for the
documentation may not know where to start.  Fix this.

Also, remove the mention of browserd from the description; the
functionality will remain, but this is no longer one of the main selling
points of the package.

Eliminate trailing whitespace from lines

Add trailing newline to Fremantle continuous mode warning message

Treat failure to acquire osso_browser name on system bus as non-fatal

This makes testing on regular Linux systems (where the system bus is
restricted to privileged processes) easier.

Listen on D-Bus system bus as well as session bus

Discussion on https://bugs.maemo.org/show_bug.cgi?id=8369 (Hermes
doesn't work when Browser Switchboard is installed) revealed that
MicroB listens on the D-Bus system bus as well as the session bus.
While it's not appropriate for applications to request opening a
browser via the system bus (which is supposed to be for requesting
system services, not applications), the fact that MicroB listens
there means that we need to too.

This fixes Browser Switchboard bug 5910.  Thanks to Andrew Flegg and
Marcin Juszkiewicz for providing the information needed to track down
this bug.

Remove continuous_mode config option from Fremantle UI

Now that we force continuous_mode on for Fremantle, there's no point
in having the setting exposed in the UI.  Remove it.

Force continuous_mode on on Fremantle

For Fremantle, we have to have continuous_mode on in order to keep
listening to the D-Bus system bus like MicroB does.  Since several
applications appear to depend on this behavior, force continuous_mode
on for Fremantle and emit a warning if the config file has it turned
off.

Make continuous mode operation the default

Continuous mode is better-tested at this point, and the ~100 KB
memory footprint isn't an issue on Maemo devices.  Moreover, a full
fix for bug 5910 (listen on D-Bus system bus as well as session bus)
on Fremantle requires continuous mode.

Remove unnecessary break statement in swb_config_load_option()

Simplify control flow in swb_config_load_option()

Just return out of the loop when a match is found, and get rid of the
retval variable.

Fix memory leak when repeated options are encountered in config file

Caught by valgrind.

Fix a broken merge

Update debian/changelog, bump version number

Merge commit 'diablo-package-3.3b1-1' into fremantle-package

Revert "Add preliminary support for Opera Mobile"

This reverts commit 81d2b2485ca2a490af7f4bb5d4e2995c35c12915.

Update debian/changelog, bump version number

Add command-line utility to Makefile install target

Add command-line utility to Makefile install target

Update debian/rules to build command-line config utility

Merge commit 'v3.3b1' into diablo-package

Restore webpages

This reverts commit 8cb71fe8a77dc9a723232c815ee4457da3fb4103.

Remove webpages for release

Revert "Add preliminary support for Opera Mobile"

This reverts commit 81d2b2485ca2a490af7f4bb5d4e2995c35c12915.

Config UI: leave other_browser_cmd unset instead of configuring an empty value

It used to be that when the user cleared the value of other_browser_cmd
in the UI, the setting would be removed from the config file instead of
writing an empty value.  Restore this behavior.

Fix check for whether a browser is installed

The intent was to exempt MicroB (and any other browser listed with
browser->binary = NULL) from the check, but the tests here didn't do
that.  Fix this so that default_browser = microb doesn't emit a warning.

Command-line utility: avoid NULL dereference when displaying string config settings

Update README and Changelog

Fix memory leak on reconfiguration

Caught by valgrind.

Describe MicroB slightly more verbosely in the UI

Emanuele Cassioli pointed out that users could potentially be unfamiliar
with the name "MicroB", since it's not used in normally user-visible
places.  Rename the entry "MicroB (stock browser)" to hopefully help
these users; better suggestions welcome.

Don't use a browser as the default browser if it's not installed

Add locations for the known browsers' binaries, and check to see if the
binary is installed before using a browser as the default.

Get rid of boilerplate code in swb_config_free()

We can loop over swb_config_options[] instead of identifying every
string config option explicitly here.

Avoid structure copies when looping through swb_config_options

Remove use_other_browser_cmd()

use_other_browser_cmd() is only called from one place now, so just move
its code there and get rid of the function.

Use an array of browser launchers instead of boilerplate code in update_default_browser()

Replace the existing set of if-elses comparing default_browser against
string constants with a loop over an array of browser launchers, which
will be (marginally) easier to maintain.

Update webpage

Merge branch 'diablo-package' into fremantle-package

Update package description

Update debian/changelog, bump version number

Merge branch 'diablo-package' into fremantle-package

Update debian/changelog, bump version number

Add preliminary support for Opera Mobile

We do this the lazy way, using use_other_browser_cmd("opera %s").  The
Opera Labs post announcing Opera Mobile for Maemo
(http://labs.opera.com/news/2010/05/11/) says that "an extra empty tab
will get opened every time you open a link from an external
application", so I suspect we may need a different method of opening
links in an already open browser.

Add preliminary support for Opera Mobile

We do this the lazy way, using use_other_browser_cmd("opera %s").  The
Opera Labs post announcing Opera Mobile for Maemo
(http://labs.opera.com/news/2010/05/11/) says that "an extra empty tab
will get opened every time you open a link from an external
application", so I suspect we may need a different method of opening
links in an already open browser.

Don't build with -mthumb on Fremantle

Using Thumb instructions causes browser-switchboard to crash with SIGILL
on Fremantle PR1.2 devices, presumably due to some combination of kernel
and toolchain bugs, so disable the use of Thumb for now.

Thanks to Uwe Kaminski (jukey) and Fred Farber (ffarber) for providing
details on this bug.

Introduce command-line config utility

Add a simple command-line configuration utility,
browser-switchboard-config, which can be used to query or set
browser-switchboard config settings.

The primary purpose of this tool is to allow browser vendors to provide
an option in their browsers to set themselves as the default browser,
via something like this (shell-like pseudocode):
if [ -x /usr/bin/browser-switchboard-config ]; then
if [ `browser-switchboard-config -b` = "mybrowser" ]; then
display "MyBrowser is the default browser."
else
button "Make MyBrowser the default browser" action "browser-switchboard-config -s -b mybrowser"
fi
else
display "Install Browser Switchboard to make MyBrowser the default browser"
fi
Browser vendors: setting yourself as the default browser without
prompting the user first (via package maintainer scripts, for example)
is strongly discouraged.

Indentation fixes

Clean up configuration again

Commit ec8b58af... ("Refactor configuration") still has too much
boilerplate code and leaves too many places that have to be modified to
add a new config option.

To fix this, add a new array of config options with name and type of
information stored, and then add an array of pointers to the elements of
struct swb_config with indices matching the list of config options to
struct swb_config.  This consolidates all the work needed to add new
config options to config.h and config.c, and allows us to replace the
boilerplate code with loops over the array of config options.

Refactor configuration

Most of the code for loading configuration from the config file was
being duplicated between the UI and the main browser-switchboard code;
also, the forthcoming command-line configuration tool should share the
code for loading and saving configuration with the UI.  Therefore,
move code for loading and saving configuration information into
functions generic enough to be shared between the UIs and
browser-switchboard.

Add preliminary support for Opera Mobile

We do this the lazy way, using use_other_browser_cmd("opera %s").  The
Opera Labs post announcing Opera Mobile for Maemo
(http://labs.opera.com/news/2010/05/11/) says that "an extra empty tab
will get opened every time you open a link from an external
application", so I suspect we may need a different method of opening
links in an already open browser.

Fremantle: clear hildon-control-panel's cached image on remove

On Fremantle, hildon-control-panel creates a screenshot of itself, which
is displayed during startup to create the illusion of a faster startup
time (sigh).  Of course, this cache doesn't appear to be regenerated
when a control panel icon is removed
(https://bugs.maemo.org/show_bug.cgi?id=9512).  Wonderful.

Work around by removing the cached image in the postrm.

Whitespace cleanups

Avoid use of Fremantle codename alone, for clarity

Update webpage

Note that on Fremantle, MicroB browser windows should be closed before installing

Update debian/changelog, bump version number

Add a postinst to kill off prestarted MicroB browser process

On Fremantle, a MicroB browser process is prestarted by default on
device startup.  This browser process takes com.nokia.osso_browser,
which means that link requests will not launch browser-switchboard
unless we kill off this browser process or restart the device.

Add a Fremantle-specific microb.desktop

MicroB's icon has a different name in Fremantle, so we need to modify
the .desktop file that we ship.

Build fremantle targets in debian/rules, clean up old unused cruft

Merge commit 'diablo-package-3.2-1' into test-branch

Conflicts:

debian/changelog
debian/rules

Revert "Initial support for launching MicroB on Fremantle"

This reverts commit f4665fd53d5c0e352e36326efee27e6ce6edce1d.

Revert "Remove some unnecessary #includes that crept in"

This reverts commit f6c9c1bcdb9a50cff459d69feeab1caa909f098c.

Revert "Zap one more extraneous #include from Fremantle MicroB work"

This reverts commit d39042804d1ee8b5f57dd5c1af884ea4e208b30a.