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

Side by Side Diff: src/client/linux/handler/microdump_extra_info.h

Issue 1334473003: Add GPU fingerprint information to breakpad microdumps. (Closed) Base URL: https://chromium.googlesource.com/breakpad/breakpad.git@master
Patch Set: Resolve TODO. Created 5 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
1 // Copyright (c) 2007, Google Inc. 1 // Copyright 2015 Google Inc.
2 // All rights reserved. 2 // All rights reserved.
3 // 3 //
4 // Redistribution and use in source and binary forms, with or without 4 // Redistribution and use in source and binary forms, with or without
5 // modification, are permitted provided that the following conditions are 5 // modification, are permitted provided that the following conditions are
6 // met: 6 // met:
7 // 7 //
8 // * Redistributions of source code must retain the above copyright 8 // * Redistributions of source code must retain the above copyright
9 // notice, this list of conditions and the following disclaimer. 9 // notice, this list of conditions and the following disclaimer.
10 // * Redistributions in binary form must reproduce the above 10 // * Redistributions in binary form must reproduce the above
11 // copyright notice, this list of conditions and the following disclaimer 11 // copyright notice, this list of conditions and the following disclaimer
12 // in the documentation and/or other materials provided with the 12 // in the documentation and/or other materials provided with the
13 // distribution. 13 // distribution.
14 // * Neither the name of Google Inc. nor the names of its 14 // * Neither the name of Google Inc. nor the names of its
15 // contributors may be used to endorse or promote products derived from 15 // contributors may be used to endorse or promote products derived from
16 // this software without specific prior written permission. 16 // this software without specific prior written permission.
17 // 17 //
18 // THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS 18 // THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
19 // "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT 19 // "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
20 // LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR 20 // LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
21 // A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT 21 // A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
22 // OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, 22 // OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
23 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT 23 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
24 // LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, 24 // LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
25 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY 25 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
26 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT 26 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
27 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE 27 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
28 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 28 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
29 29
30 // ./dump_syms dump_syms_regtest.pdb > dump_syms_regtest.sym 30 #ifndef CLIENT_LINUX_HANDLER_MICRODUMP_EXTRA_INFO_H_
31 #define CLIENT_LINUX_HANDLER_MICRODUMP_EXTRA_INFO_H_
31 32
32 namespace google_breakpad { 33 namespace google_breakpad {
33 34
34 class C { 35 struct MicrodumpExtraInfo {
35 public: 36 const char* build_fingerprint;
36 C() : member_(1) {} 37 const char* product_info;
37 virtual ~C() {} 38 const char* gpu_fingerprint;
38 39
39 void set_member(int value) { member_ = value; } 40 MicrodumpExtraInfo()
40 int member() const { return member_; } 41 : build_fingerprint(NULL), product_info(NULL), gpu_fingerprint(NULL) {}
41 42
42 void f() { member_ = g(); } 43 MicrodumpExtraInfo(const char* build_fingerprint_val,
Primiano Tucci (use gerrit) 2015/09/10 12:38:58 please add a comment clarifying that all the const
Tobias Sargeant 2015/09/10 13:18:38 Done.
43 virtual int g() { return 2; } 44 const char* product_info_val,
44 static char* h(const C &that) { return 0; } 45 const char* gpu_fingerprint_val)
45 46 : build_fingerprint(build_fingerprint_val),
46 private: 47 product_info(product_info_val),
47 int member_; 48 gpu_fingerprint(gpu_fingerprint_val) {}
48 }; 49 };
49 50
50 static int i() {
51 return 3;
52 } 51 }
53 52
54 } // namespace google_breakpad 53 #endif // CLIENT_LINUX_HANDLER_MICRODUMP_EXTRA_INFO_H_
55
56 int main(int argc, char **argv) {
57 google_breakpad::C object;
58 object.set_member(google_breakpad::i());
59 object.f();
60 int value = object.g();
61 char *nothing = object.h(object);
62
63 return 0;
64 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698