Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(282)

Side by Side Diff: third_party/WebKit/Source/devtools/front_end/common/BrowserVersionInfo.js

Issue 2300403003: DevTools: patch browser's Chrome version into Chrome user agents for emulation (Closed)
Patch Set: Removed old stuff Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(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 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698