Re: [PATCH] Support for launching Chrome browser(if present) from HS20 client

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



HI Johannes,

The code already uses /system/bin/am start -a
android.intent.action.VIEW [...] .. as suggested by you. Not
specifying the -n argument will give the option to the user to select
the browser. But it pops out a dialog for the user to select the
browser, which is an extra step and might irritate the user also.

On Tue, Nov 3, 2015 at 4:20 PM, Somdas Bandyopadhyay <som6000@xxxxxxxxx> wrote:
> Yes, that is also an option (to do that we have to remove the -n
> argument in the am command). But that will pop out a dialog for the
> user to select the preferred browser. From an UX perspective that
> might not be good, as all of a sudden the user will get an pop up to
> select a browser. Currently the code always launches the Android
> Native browser.  I added the support to launch chrome browser if it is
> present. Else it continues with the normal flow of launching native
> browser. I feel that's a better option.
>
>
>
> On Tue, Nov 3, 2015 at 4:04 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote:
>> On Tue, 2015-11-03 at 15:49 +0530, Somdas Bandyopadhyay wrote:
>>> PFA the patch file.
>>>
>>> If Chrome browser is present in the device then launch Chrome
>>> browser.
>>> Else launch the normal native browser (original flow). PFA the patch
>>> file.
>>>
>> That seems really odd - wouldn't you use the intents framework to
>> launch the user's default browser?
>>
>> johannes
>
>
>
> --
> Thanks and regards,
> Somdas



-- 
Thanks and regards,
Somdas

_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap



[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux