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

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

Issue 2561963002: base: Remove the string logging from CHECK(). (Closed)
Patch Set: checkstring: rebase Created 4 years 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" 7 #include "base/logging.h"
8 8
9 namespace extensions { 9 namespace extensions {
10 10
11 NativeHandler::NativeHandler() : is_valid_(true) {} 11 NativeHandler::NativeHandler() : is_valid_(true) {}
12 12
13 NativeHandler::~NativeHandler() { 13 NativeHandler::~NativeHandler() {
14 CHECK(!is_valid_) << "NativeHandlers must be invalidated before destruction"; 14 // NativeHandlers must be invalidated before destruction
15 CHECK(!is_valid_);
15 } 16 }
16 17
17 void NativeHandler::Invalidate() { 18 void NativeHandler::Invalidate() {
18 CHECK(is_valid_); 19 CHECK(is_valid_);
19 is_valid_ = false; 20 is_valid_ = false;
20 } 21 }
21 22
22 } // namespace extensions 23 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698