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

Side by Side Diff: extensions/renderer/native_handler.cc

Issue 1074273002: Move the event attach/detach logic on unload from event.js to (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: being process of relanding - more test assertions Created 5 years, 8 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 #include "extensions/renderer/native_handler.h" 5 #include "extensions/renderer/native_handler.h"
6 6
7 #include "base/logging.h"
8
7 namespace extensions { 9 namespace extensions {
8 10
9 NativeHandler::NativeHandler() : is_valid_(true) {} 11 NativeHandler::NativeHandler() : is_valid_(true) {}
10 12
11 NativeHandler::~NativeHandler() {} 13 NativeHandler::~NativeHandler() {
14 CHECK(!is_valid_) << "NativeHandlers must be invalidated before destruction";
15 }
12 16
13 void NativeHandler::Invalidate() { is_valid_ = false; } 17 void NativeHandler::Invalidate() {
18 CHECK(is_valid_);
19 is_valid_ = false;
20 }
14 21
15 } // namespace extensions 22 } // namespace extensions
OLDNEW
« no previous file with comments | « extensions/renderer/native_handler.h ('k') | extensions/renderer/object_backed_native_handler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698