Chromium Code Reviews| OLD | NEW |
|---|---|
| 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() {} |
| OLD | NEW |