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

Side by Side Diff: content/browser/service_worker/service_worker_info.h

Issue 691413002: Removing redundant 'is_null' field in ServiceWorkerVersionInfo (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 #ifndef CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_INFO_H_ 5 #ifndef CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_INFO_H_
6 #define CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_INFO_H_ 6 #define CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_INFO_H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "content/browser/service_worker/service_worker_version.h" 10 #include "content/browser/service_worker/service_worker_version.h"
11 #include "content/common/content_export.h" 11 #include "content/common/content_export.h"
12 #include "url/gurl.h" 12 #include "url/gurl.h"
13 13
14 namespace content { 14 namespace content {
15 15
16 class CONTENT_EXPORT ServiceWorkerVersionInfo { 16 class CONTENT_EXPORT ServiceWorkerVersionInfo {
17 public: 17 public:
18 ServiceWorkerVersionInfo(); 18 ServiceWorkerVersionInfo();
19 ServiceWorkerVersionInfo(ServiceWorkerVersion::RunningStatus running_status, 19 ServiceWorkerVersionInfo(ServiceWorkerVersion::RunningStatus running_status,
20 ServiceWorkerVersion::Status status, 20 ServiceWorkerVersion::Status status,
21 const GURL& script_url, 21 const GURL& script_url,
22 int64 version_id, 22 int64 version_id,
23 int process_id, 23 int process_id,
24 int thread_id, 24 int thread_id,
25 int devtools_agent_route_id); 25 int devtools_agent_route_id);
26 ~ServiceWorkerVersionInfo(); 26 ~ServiceWorkerVersionInfo();
27 27
28 bool is_null;
29 ServiceWorkerVersion::RunningStatus running_status; 28 ServiceWorkerVersion::RunningStatus running_status;
30 ServiceWorkerVersion::Status status; 29 ServiceWorkerVersion::Status status;
31 GURL script_url; 30 GURL script_url;
32 int64 version_id; 31 int64 version_id;
33 int process_id; 32 int process_id;
34 int thread_id; 33 int thread_id;
35 int devtools_agent_route_id; 34 int devtools_agent_route_id;
36 }; 35 };
37 36
38 class CONTENT_EXPORT ServiceWorkerRegistrationInfo { 37 class CONTENT_EXPORT ServiceWorkerRegistrationInfo {
(...skipping 11 matching lines...) Expand all
50 int64 registration_id; 49 int64 registration_id;
51 ServiceWorkerVersionInfo controlling_version; 50 ServiceWorkerVersionInfo controlling_version;
52 ServiceWorkerVersionInfo active_version; 51 ServiceWorkerVersionInfo active_version;
53 ServiceWorkerVersionInfo waiting_version; 52 ServiceWorkerVersionInfo waiting_version;
54 ServiceWorkerVersionInfo installing_version; 53 ServiceWorkerVersionInfo installing_version;
55 }; 54 };
56 55
57 } // namespace content 56 } // namespace content
58 57
59 #endif // CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_INFO_H_ 58 #endif // CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_INFO_H_
OLDNEW
« no previous file with comments | « content/browser/service_worker/service_worker_browsertest.cc ('k') | content/browser/service_worker/service_worker_info.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698