Lgtm. we'll try to support it. Yongsheng
> -----Original Message----- > From: Kenneth Rohde Christiansen [mailto:[email protected]] > Sent: Monday, December 02, 2013 4:19 PM > To: Zhu, Yongsheng > Cc: Huo, Halton; Ramakrishnan, Girish; Menard, Alexis; > <[email protected]> > Subject: Re: [Crosswalk-dev] User Agent of Crosswalk. > > We should probably append "Mobile Crosswalk" on Tizen Mobile at least and on > Android, if it doesnt have Mobile already > > On Mon, Dec 2, 2013 at 6:48 AM, Zhu, Yongsheng <[email protected]> > wrote: > > Thanks for your reply, Halton. > > I've created one issue to track this: > > https://crosswalk-project.org/jira/browse/XWALK-595 > > > > Yongsheng > > > > > >> -----Original Message----- > >> From: Huo, Halton > >> Sent: Monday, December 02, 2013 1:45 PM > >> To: Zhu, Yongsheng; Ramakrishnan, Girish; Menard, Alexis > >> Cc: <[email protected]> > >> Subject: RE: [Crosswalk-dev] User Agent of Crosswalk. > >> > >> According > >> > https://developers.google.com/chrome/mobile/docs/webview/overview#wha > >> t _is_the_default_user-agent, I vote +1 for appending > >> "Crosswalk/<Crosswalk > >> Ver>" at the tail. > >> > >> > -----Original Message----- > >> > From: Crosswalk-dev > >> > [mailto:[email protected]] On > >> > Behalf Of Zhu, Yongsheng > >> > Sent: Monday, December 02, 2013 1:00 PM > >> > To: Ramakrishnan, Girish; Menard, Alexis > >> > Cc: <[email protected]> > >> > Subject: Re: [Crosswalk-dev] User Agent of Crosswalk. > >> > > >> > > >> Below is the current UA string for Android: > >> > > >> Mozilla/5.0 (Linux; Android 4.2.2; ZTE V975 Build/JDQ39) > >> > > >>AppleWebKit/537.36 (KHTML, like Gecko) Chrome/3.31.33.0 Mobile > >> > > >>Safari/537.36 > >> > I suggest to append ''Crosswalk/3.xx.xx.x" at the end of the above > >> > user agent string for Android. > >> > All, any comments? > >> > > >> > Yongsheng > >> > > >> > > -----Original Message----- > >> > > From: Ramakrishnan, Girish > >> > > Sent: Saturday, November 23, 2013 7:33 AM > >> > > To: Menard, Alexis; Zhu, Yongsheng > >> > > Cc: <[email protected]> > >> > > Subject: Re: [Crosswalk-dev] User Agent of Crosswalk. > >> > > > >> > > Just following this up. Did we change the UA? If so, what did we > >> > > change it > >> > to? > >> > > > >> > > Girish > >> > > > >> > > On 11/11/13, 3:23 AM, "Menard, Alexis" <[email protected]> > >> > wrote: > >> > > > >> > > > > >> > > > > >> > > >On Nov 11, 2013, at 12:32 AM, Zhu, Yongsheng > >> > > ><[email protected]> > >> > > >wrote: > >> > > > > >> > > >> I suggest to at least use Chrome mobile user agent so that web > >> > > >>pages can be aware and send mobile version web pages. > >> > > >> Adding the crosswalk information is acceptable and makes the > >> > > >>UA longer but to be honest we need the support of web servers > >> > > >>in > >> > future. > >> > > > > >> > > >Indeed but we need to add Crosswalk so you can actually detec it > >> > > >over the < raw > Android. > >> > > > > >> > > >> > >> > > >> Below is the current UA string for Android: > >> > > >> Mozilla/5.0 (Linux; Android 4.2.2; ZTE V975 Build/JDQ39) > >> > > >>AppleWebKit/537.36 (KHTML, like Gecko) Chrome/3.31.33.0 Mobile > >> > > >>Safari/537.36 > >> > > >> > >> > > >> Yongsheng > >> > > >> > >> > > >>> -----Original Message----- > >> > > >>> From: Crosswalk-dev > >> > > >>>[mailto:[email protected]] > >> > > >>> On Behalf Of Menard, Alexis > >> > > >>> Sent: Saturday, November 09, 2013 1:03 AM > >> > > >>> To: <[email protected]> > >> > > >>> Subject: [Crosswalk-dev] User Agent of Crosswalk. > >> > > >>> > >> > > >>> Hi, > >> > > >>> > >> > > >>> We didn't really define the user agent of Crosswalk so far. > >> > > >>>We all know that user-agent are evil, blablabla, however we > >> > > >>>can't force the entire planet to follow good coding > >> > > >>>practices. Today it's defined like that : > >> > > >>> > >> > > >>> std::string XWalkContentClient::GetUserAgent() const { // > >> > > >>> TODO(hmin): Define user agent for xwalk. > >> > > >>> std::string product = "Chrome/" XWALK_VERSION; > >> > > >>> CommandLine* command_line = > >> > CommandLine::ForCurrentProcess(); if > >> > > >>> (command_line->HasSwitch(switches::kUseMobileUserAgent)) > >> > > >>> product += " Mobile"; > >> > > >>> return webkit_glue::BuildUserAgentFromProduct(product); > >> > > >>> } > >> > > >>> > >> > > >>> where XWALK_VERSION is for example 1.29.4.0. > >> > > >>> > >> > > >>> I believe we should probably put < crosswalk > somewhere no? > >> > > >>> > >> > > >>> Ideas? > >> > > >>> > >> > > >>> Software Engineer @ > >> > > >>> Intel Open Source Technology Center > >> > > >>> > >> > > >>> _______________________________________________ > >> > > >>> Crosswalk-dev mailing list > >> > > >>> [email protected] > >> > > >>> https://lists.crosswalk-project.org/mailman/listinfo/crosswal > >> > > >>> k-d > >> > > >>> ev > >> > > > > >> > > >_______________________________________________ > >> > > >Crosswalk-dev mailing list > >> > > >[email protected] > >> > > >https://lists.crosswalk-project.org/mailman/listinfo/crosswalk-d > >> > > >ev > >> > > >> > _______________________________________________ > >> > Crosswalk-dev mailing list > >> > [email protected] > >> > https://lists.crosswalk-project.org/mailman/listinfo/crosswalk-dev > > _______________________________________________ > > Crosswalk-dev mailing list > > [email protected] > > https://lists.crosswalk-project.org/mailman/listinfo/crosswalk-dev > > > > -- > Kenneth Rohde Christiansen > Web Platform Architect, Intel Corporation. > Phone +45 4294 9458 ﹆﹆﹆ _______________________________________________ Crosswalk-dev mailing list [email protected] https://lists.crosswalk-project.org/mailman/listinfo/crosswalk-dev
