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

Side by Side Diff: chrome/browser/android/webapk/webapk_info.cc

Issue 2714633003: Adds more metadata to the about:webapks page (Closed)
Patch Set: Adds more metadata to the about:webapks page Created 3 years, 10 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
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/android/webapk/webapk_info.h" 5 #include "chrome/browser/android/webapk/webapk_info.h"
dominickn 2017/02/27 02:27:02 #include <utility> for std::move()
gonzalon 2017/02/27 16:00:12 Done.
6 6
7 WebApkInfo::WebApkInfo(std::string short_name, 7 WebApkInfo::WebApkInfo(std::string name,
8 std::string short_name,
8 std::string package_name, 9 std::string package_name,
9 int shell_apk_version, 10 int shell_apk_version,
10 int version_code) 11 int version_code,
11 : short_name(short_name), 12 std::string uri,
13 std::string scope,
14 std::string manifest_url,
15 std::string manifest_start_url,
16 blink::WebDisplayMode display,
17 blink::WebScreenOrientationLockType orientation,
18 int64_t theme_color,
19 int64_t background_color)
20 : name(name),
dominickn 2017/02/27 02:27:02 Every string in here should have std::move(), i.e.
gonzalon 2017/02/27 16:00:12 Done.
21 short_name(short_name),
12 package_name(package_name), 22 package_name(package_name),
13 shell_apk_version(shell_apk_version), 23 shell_apk_version(shell_apk_version),
14 version_code(version_code) {} 24 version_code(version_code),
25 uri(uri),
26 scope(scope),
27 manifest_url(manifest_url),
28 manifest_start_url(manifest_start_url),
29 display(display),
30 orientation(orientation),
31 theme_color(theme_color),
32 background_color(background_color) {}
15 33
16 WebApkInfo::~WebApkInfo() {} 34 WebApkInfo::~WebApkInfo() {}
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698