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

Unified Diff: chrome/browser/extensions/api/declarative/declarative_rule.h

Issue 16703018: Rewrite scoped_ptr<T>(NULL) to use the default ctor in chrome/. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 6 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/extensions/api/declarative/declarative_rule.h
diff --git a/chrome/browser/extensions/api/declarative/declarative_rule.h b/chrome/browser/extensions/api/declarative/declarative_rule.h
index bdc36fef0b00fe954fb16ff0942f866c7e5862ad..11d223ab733c246b519dd45b109870761d97960f 100644
--- a/chrome/browser/extensions/api/declarative/declarative_rule.h
+++ b/chrome/browser/extensions/api/declarative/declarative_rule.h
@@ -299,7 +299,7 @@ DeclarativeConditionSet<ConditionT>::Create(
scoped_ptr<ConditionT> condition =
ConditionT::Create(url_matcher_condition_factory, **i, error);
if (!error->empty())
- return scoped_ptr<DeclarativeConditionSet>(NULL);
+ return scoped_ptr<DeclarativeConditionSet>();
result.push_back(make_linked_ptr(condition.release()));
}
@@ -359,7 +359,7 @@ DeclarativeActionSet<ActionT>::Create(
scoped_refptr<const ActionT> action =
ActionT::Create(**i, error, bad_message);
if (!error->empty() || *bad_message)
- return scoped_ptr<DeclarativeActionSet>(NULL);
+ return scoped_ptr<DeclarativeActionSet>();
result.push_back(action);
}

Powered by Google App Engine
This is Rietveld 408576698