Unified Diff: templates/chromeInfo.js.tmpl
Issue 29454678:
Issue 5085 - Add edgeInfo.js template for edge specific builds (Closed)
Patch Set: readd chrome logic, remove more chrome specific code from edge tmpl
Created June 7, 2017, 1:08 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
templates/edgeInfo.js.tmpl » ('j')
|
templates/edgeInfo.js.tmpl » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: templates/chromeInfo.js.tmpl |
| =================================================================== |
| --- a/templates/chromeInfo.js.tmpl |
| +++ b/templates/chromeInfo.js.tmpl |
| @@ -16,23 +16,16 @@ |
| { |
|
Sebastian Noack
2017/06/07 13:26:12
The "platform" variable defined above isn't necess
Jon Sonesen
2017/06/07 13:56:07
Acknowledged.
|
| let app = match[1]; |
| let ver = match[2]; |
| if (app == "Chrome") |
| { |
| platformVersion = ver; |
| } |
| - else if (app == "Edge") |
| - { |
| - platform = "edgehtml"; |
| - platformVersion = ver; |
| - application = "edge"; |
| - applicationVersion = "0"; |
| - } |
| else if (app != "Mozilla" && app != "AppleWebKit" && app != "Safari") |
| { |
| // For compatibility with legacy websites, Chrome's UA |
| // also includes a Mozilla, AppleWebKit and Safari token. |
| // Any further name/version pair indicates a fork. |
| application = app == "OPR" ? "opera" : app.toLowerCase(); |
| applicationVersion = ver; |
| } |
« no previous file with comments
|
« no previous file
|
templates/edgeInfo.js.tmpl » ('j')
|
templates/edgeInfo.js.tmpl » ('J')
