Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 /* | 1 /* |
| 2 * Copyright (C) 2010 Google Inc. All rights reserved. | 2 * Copyright (C) 2010 Google Inc. 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 | 5 * modification, are permitted provided that the following conditions |
| 6 * are met: | 6 * are met: |
| 7 * | 7 * |
| 8 * 1. Redistributions of source code must retain the above copyright | 8 * 1. 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 * 2. Redistributions in binary form must reproduce the above copyright | 10 * 2. Redistributions in binary form must reproduce the above copyright |
| (...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 69 | 69 |
| 70 void didOpenDatabase(PassRefPtrWillBeRawPtr<Database>, const String& domain, const String& name, const String& version); | 70 void didOpenDatabase(PassRefPtrWillBeRawPtr<Database>, const String& domain, const String& name, const String& version); |
| 71 private: | 71 private: |
| 72 explicit InspectorDatabaseAgent(); | 72 explicit InspectorDatabaseAgent(); |
| 73 | 73 |
| 74 Database* databaseForId(const String& databaseId); | 74 Database* databaseForId(const String& databaseId); |
| 75 InspectorDatabaseResource* findByFileName(const String& fileName); | 75 InspectorDatabaseResource* findByFileName(const String& fileName); |
| 76 | 76 |
| 77 InspectorFrontend::Database* m_frontend; | 77 InspectorFrontend::Database* m_frontend; |
| 78 typedef WillBePersistentHeapHashMap<String, RefPtrWillBeMember<InspectorData baseResource> > DatabaseResourcesHeapMap; | 78 typedef WillBePersistentHeapHashMap<String, RefPtrWillBeMember<InspectorData baseResource> > DatabaseResourcesHeapMap; |
| 79 GC_PLUGIN_IGNORE_CYCLE | |
|
haraken
2014/03/21 10:31:40
Do we need to add this? I mean, isn't it enough to
zerny-chromium
2014/03/21 10:46:19
This is the persistent handle that causes the leak
| |
| 79 DatabaseResourcesHeapMap m_resources; | 80 DatabaseResourcesHeapMap m_resources; |
| 80 bool m_enabled; | 81 bool m_enabled; |
| 81 }; | 82 }; |
| 82 | 83 |
| 83 } // namespace WebCore | 84 } // namespace WebCore |
| 84 | 85 |
| 85 #endif // !defined(InspectorDatabaseAgent_h) | 86 #endif // !defined(InspectorDatabaseAgent_h) |
| OLD | NEW |