summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorjhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-10-12 03:55:15 +0000
committerjhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-10-12 03:55:15 +0000
commite4f1acf6c2fad00621a7e3d589eb3f33e6889a9b (patch)
tree1a7a42ea973e0f7aea34d24364d55c6dfb68f342
parentf5ffecf35b670b1d10b3a27f6a408af5c187413c (diff)
downloadchromium_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.cc22
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);