Chromium Code Reviews| OLD | NEW |
|---|---|
| (Empty) | |
| 1 // Copyright 2016 The Chromium Authors. All rights reserved. | |
| 2 // Use of this source code is governed by a BSD-style license that can be | |
| 3 // found in the LICENSE file. | |
| 4 | |
| 5 WebInspector.BrowserVersionInfo = { | |
| 6 DefaultChromeVersion: "54.0.2834.0", | |
|
dgozman
2016/09/07 22:06:28
Let's have this in a default CurrentBrowserVersion
luoe
2016/09/07 22:46:32
Done.
| |
| 7 | |
| 8 _version: "54.0.2834.0", | |
| 9 | |
| 10 /** | |
| 11 * @param {string} uaString | |
| 12 * @return {string} | |
| 13 */ | |
| 14 patchUserAgentWithCurrentVersion: function(uaString) | |
| 15 { | |
| 16 if (uaString.indexOf("%s") === -1) | |
| 17 return uaString; | |
| 18 | |
| 19 var currentVersion = WebInspector.BrowserVersionInfo._version; | |
| 20 if (currentVersion[0] === "@") | |
| 21 currentVersion = WebInspector.BrowserVersionInfo.DefaultChromeVersio n; | |
| 22 console.assert(uaString.indexOf("%s") === uaString.lastIndexOf("%s"), "U ser agent string contains multiple %s"); | |
| 23 return String.sprintf(uaString, currentVersion); | |
|
dgozman
2016/09/07 22:06:28
I think instead of replacing the version inside, w
luoe
2016/09/07 22:46:32
I don't think we have sufficient information to ge
| |
| 24 } | |
| 25 }; | |
| OLD | NEW |