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

Side by Side Diff: plugin/cross/plugin_logging.h

Issue 4957002: - Check for a NULL PluginObject at all NPAPI entry points. This fixes a crash in Chrome where ... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/o3d/
Patch Set: '' Created 10 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 | Annotate | Revision Log
« no previous file with comments | « plugin/cross/main.cc ('k') | plugin/linux/main_linux.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2009, Google Inc. 2 * Copyright 2009, Google Inc.
3 * All rights reserved. 3 * All rights reserved.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions are 6 * modification, are permitted provided that the following conditions are
7 * met: 7 * met:
8 * 8 *
9 * * Redistributions of source code must retain the above copyright 9 * * Redistributions of source code must retain the above copyright
10 * notice, this list of conditions and the following disclaimer. 10 * notice, this list of conditions and the following disclaimer.
(...skipping 20 matching lines...) Expand all
31 31
32 32
33 // This file defines the logging object which performs the metric aggregation 33 // This file defines the logging object which performs the metric aggregation
34 // and uploading. This class takes care of the initialization of the logging 34 // and uploading. This class takes care of the initialization of the logging
35 // object and determining if the user has opted in or out to having logs sent 35 // object and determining if the user has opted in or out to having logs sent
36 // back. Furthermore, there are some helper functions to make testing easier. 36 // back. Furthermore, there are some helper functions to make testing easier.
37 37
38 #ifndef O3D_PLUGIN_CROSS_PLUGIN_LOGGING_H_ 38 #ifndef O3D_PLUGIN_CROSS_PLUGIN_LOGGING_H_
39 #define O3D_PLUGIN_CROSS_PLUGIN_LOGGING_H_ 39 #define O3D_PLUGIN_CROSS_PLUGIN_LOGGING_H_
40 40
41 #include "statsreport/metrics.h"
41 #include "statsreport/common/highres_timer.h" 42 #include "statsreport/common/highres_timer.h"
42 #include "base/scoped_ptr.h" 43 #include "base/scoped_ptr.h"
43 #include "base/basictypes.h" 44 #include "base/basictypes.h"
44 45
45 namespace o3d { 46 namespace o3d {
46 47
47 class PluginLogging { 48 class PluginLogging {
48 public: 49 public:
49 PluginLogging(); 50 PluginLogging();
50 virtual ~PluginLogging() {} 51 virtual ~PluginLogging() {}
(...skipping 97 matching lines...) Expand 10 before | Expand all | Expand 10 after
148 scoped_ptr<HighresTimer> timer_; 149 scoped_ptr<HighresTimer> timer_;
149 uint64 running_time_; 150 uint64 running_time_;
150 uint64 prev_uptime_seconds_; 151 uint64 prev_uptime_seconds_;
151 uint64 prev_cputime_seconds_; 152 uint64 prev_cputime_seconds_;
152 DISALLOW_COPY_AND_ASSIGN(PluginLogging); 153 DISALLOW_COPY_AND_ASSIGN(PluginLogging);
153 }; 154 };
154 155
155 } // namespace o3d 156 } // namespace o3d
156 157
157 #endif // O3D_PLUGIN_CROSS_PLUGIN_LOGGING_H_ 158 #endif // O3D_PLUGIN_CROSS_PLUGIN_LOGGING_H_
OLDNEW
« no previous file with comments | « plugin/cross/main.cc ('k') | plugin/linux/main_linux.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698