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

Unified Diff: webkit/common/dom_storage/dom_storage_map.cc

Issue 17327004: Replace base::NullableString16(bool) usage with default constructor. (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: webkit/common/dom_storage/dom_storage_map.cc
diff --git a/webkit/common/dom_storage/dom_storage_map.cc b/webkit/common/dom_storage/dom_storage_map.cc
index 377d185841afb3bc2df33db7301af5d8c8b72fe4..5cc3c5e62fd10086db998e6e3cd134a149f26461 100644
--- a/webkit/common/dom_storage/dom_storage_map.cc
+++ b/webkit/common/dom_storage/dom_storage_map.cc
@@ -41,7 +41,7 @@ unsigned DomStorageMap::Length() const {
base::NullableString16 DomStorageMap::Key(unsigned index) {
if (index >= values_.size())
- return base::NullableString16(true);
+ return base::NullableString16();
while (last_key_index_ != index) {
if (last_key_index_ > index) {
--key_iterator_;
@@ -57,7 +57,7 @@ base::NullableString16 DomStorageMap::Key(unsigned index) {
base::NullableString16 DomStorageMap::GetItem(const base::string16& key) const {
ValuesMap::const_iterator found = values_.find(key);
if (found == values_.end())
- return base::NullableString16(true);
+ return base::NullableString16();
return found->second;
}
@@ -66,7 +66,7 @@ bool DomStorageMap::SetItem(
base::NullableString16* old_value) {
ValuesMap::const_iterator found = values_.find(key);
if (found == values_.end())
- *old_value = base::NullableString16(true);
+ *old_value = base::NullableString16();
else
*old_value = found->second;

Powered by Google App Engine
This is Rietveld 408576698