diff options
author | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-12 03:55:15 +0000 |
---|---|---|
committer | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-12 03:55:15 +0000 |
commit | e4f1acf6c2fad00621a7e3d589eb3f33e6889a9b (patch) | |
tree | 1a7a42ea973e0f7aea34d24364d55c6dfb68f342 | |
parent | f5ffecf35b670b1d10b3a27f6a408af5c187413c (diff) | |
download | chromium_src-e4f1acf6c2fad00621a7e3d589eb3f33e6889a9b.zip chromium_src-e4f1acf6c2fad00621a7e3d589eb3f33e6889a9b.tar.gz chromium_src-e4f1acf6c2fad00621a7e3d589eb3f33e6889a9b.tar.bz2 |
Web Intents: Remove handling of NOTREACHEDs.
BUG=none
TEST=none
R=groby@chromium.org
Review URL: http://codereview.chromium.org/8228019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105000 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r-- | chrome/browser/webdata/web_intents_table.cc | 22 |
1 files changed, 5 insertions, 17 deletions
diff --git a/chrome/browser/webdata/web_intents_table.cc b/chrome/browser/webdata/web_intents_table.cc index df7ec28..9be56b7 100644 --- a/chrome/browser/webdata/web_intents_table.cc +++ b/chrome/browser/webdata/web_intents_table.cc @@ -52,14 +52,10 @@ bool WebIntentsTable::Init() { "disposition VARCHAR," "UNIQUE (service_url, action, type))")) { NOTREACHED(); - return false; } - if (!db_->Execute("CREATE INDEX web_intents_index " - "ON web_intents (action)")) { + if (!db_->Execute("CREATE INDEX web_intents_index ON web_intents (action)")) NOTREACHED(); - return false; - } return true; } @@ -76,10 +72,8 @@ bool WebIntentsTable::GetWebIntents( sql::Statement s(db_->GetUniqueStatement( "SELECT service_url, action, type, title, disposition FROM web_intents " "WHERE action=?")); - if (!s) { + if (!s) NOTREACHED() << "Statement prepare failed"; - return false; - } s.BindString16(0, action); return ExtractIntents(&s, intents); @@ -90,10 +84,8 @@ bool WebIntentsTable::GetAllWebIntents( DCHECK(intents); sql::Statement s(db_->GetUniqueStatement( "SELECT service_url, action, type, title, disposition FROM web_intents")); - if (!s) { + if (!s) NOTREACHED() << "Statement prepare failed"; - return false; - } return ExtractIntents(&s, intents); } @@ -103,10 +95,8 @@ bool WebIntentsTable::SetWebIntent(const WebIntentServiceData& intent) { "INSERT OR REPLACE INTO web_intents " "(service_url, type, action, title, disposition) " "VALUES (?, ?, ?, ?, ?)")); - if (!s) { + if (!s) NOTREACHED() << "Statement prepare failed"; - return false; - } // Default to window disposition. string16 disposition = ASCIIToUTF16("window"); @@ -127,10 +117,8 @@ bool WebIntentsTable::RemoveWebIntent(const WebIntentServiceData& intent) { sql::Statement s(db_->GetUniqueStatement( "DELETE FROM web_intents " "WHERE service_url = ? AND action = ? AND type = ?")); - if (!s) { + if (!s) NOTREACHED() << "Statement prepare failed"; - return false; - } s.BindString(0, intent.service_url.spec()); s.BindString16(1, intent.action); |