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

Side by Side Diff: content/shell/tools/plugin/Tests/DocumentOpenInDestroyStream.cpp

Issue 671663002: Standardize usage of virtual/override/final in content/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 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 /* 5 /*
6 * Copyright (C) 2010 Apple Inc. All rights reserved. 6 * Copyright (C) 2010 Apple Inc. All rights reserved.
7 * 7 *
8 * Redistribution and use in source and binary forms, with or without 8 * Redistribution and use in source and binary forms, with or without
9 * modification, are permitted provided that the following conditions 9 * modification, are permitted provided that the following conditions
10 * are met: 10 * are met:
(...skipping 26 matching lines...) Expand all
37 37
38 class DocumentOpenInDestroyStream : public PluginTest { 38 class DocumentOpenInDestroyStream : public PluginTest {
39 public: 39 public:
40 DocumentOpenInDestroyStream(NPP npp, const string& identifier) 40 DocumentOpenInDestroyStream(NPP npp, const string& identifier)
41 : PluginTest(npp, identifier) 41 : PluginTest(npp, identifier)
42 , m_shouldOpen(true) 42 , m_shouldOpen(true)
43 { 43 {
44 } 44 }
45 45
46 private: 46 private:
47 virtual NPError NPP_DestroyStream(NPStream*, NPReason) override 47 NPError NPP_DestroyStream(NPStream*, NPReason) override {
48 {
49 if (m_shouldOpen) { 48 if (m_shouldOpen) {
50 testDocumentOpen(m_npp); 49 testDocumentOpen(m_npp);
51 m_shouldOpen = false; 50 m_shouldOpen = false;
52 } 51 }
53 52
54 return NPERR_NO_ERROR; 53 return NPERR_NO_ERROR;
55 } 54 }
56 55
57 bool m_shouldOpen; 56 bool m_shouldOpen;
58 }; 57 };
59 58
60 static PluginTest::Register<DocumentOpenInDestroyStream> documentOpenInDestroySt ream("document-open-in-destroy-stream"); 59 static PluginTest::Register<DocumentOpenInDestroyStream> documentOpenInDestroySt ream("document-open-in-destroy-stream");
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698