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

Unified Diff: chrome/browser/app_controller_mac.mm

Issue 6539002: Begun the DCHECK() cleanup. Starting off with /src/chrome/a* and /src/chrome/... (Closed) Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: '' Created 9 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/app_controller_mac.mm
===================================================================
--- chrome/browser/app_controller_mac.mm (revision 75286)
+++ chrome/browser/app_controller_mac.mm (working copy)
@@ -1118,7 +1118,7 @@
// window controller.
- (void)prefsWindowClosed:(NSNotification*)notification {
NSWindow* window = [prefsController_ window];
- DCHECK([notification object] == window);
+ DCHECK_EQ([notification object], window);
NSNotificationCenter* defaultCenter = [NSNotificationCenter defaultCenter];
[defaultCenter removeObserver:self
name:NSWindowWillCloseNotification
@@ -1171,7 +1171,7 @@
// window controller.
- (void)aboutWindowClosed:(NSNotification*)notification {
NSWindow* window = [aboutController_ window];
- DCHECK(window == [notification object]);
+ DCHECK_EQ(window, [notification object]);
[[NSNotificationCenter defaultCenter]
removeObserver:self
name:NSWindowWillCloseNotification
@@ -1239,7 +1239,7 @@
for (ExtensionList::const_iterator cursor = applications.begin();
cursor != applications.end();
++cursor, ++position) {
- DCHECK(position == applications.GetPosition(*cursor));
+ DCHECK_EQ(position, applications.GetPosition(*cursor));
NSString* itemStr =
base::SysUTF16ToNSString(UTF8ToUTF16((*cursor)->name()));
scoped_nsobject<NSMenuItem> appItem([[NSMenuItem alloc]

Powered by Google App Engine
This is Rietveld 408576698