Chrome Alarm Clear and ClearAll Updates

chrome/browser/extensions/api/alarms/alarms_api.cc

Index: chrome/browser/extensions/api/alarms/alarms_api.cc
diff --git a/chrome/browser/extensions/api/alarms/alarms_api.cc b/chrome/browser/extensions/api/alarms/alarms_api.cc
index 7c0621498b44ee8703b638f26dd7560dbb00e2d1..16aff6994efa50c7c10fbda60d623acdb3a5655c 100644
--- a/chrome/browser/extensions/api/alarms/alarms_api.cc
+++ b/chrome/browser/extensions/api/alarms/alarms_api.cc
@@ -189,10 +189,8 @@ bool AlarmsClearFunction::RunImpl() {
 }
 
 void AlarmsClearFunction::Callback(const std::string& name, bool success) {
-  if (!success)
-    error_ = ErrorUtils::FormatErrorMessage(kAlarmNotFound, name);
-
-  SendResponse(success);
+  SetResult(new base::FundamentalValue(success));
+  SendResponse(true);
 }
 
 bool AlarmsClearAllFunction::RunImpl() {
@@ -202,6 +200,7 @@ bool AlarmsClearAllFunction::RunImpl() {
 }
 
 void AlarmsClearAllFunction::Callback() {
+  SetResult(new base::FundamentalValue(true));

[not at google - send to devlin, 2014/03/20 10:52:21]

seems more correct to set this to (num_alarms_cleared > 0) if you can plumb that
through. seems trivial.

[Matt Perry, 2014/03/20 18:53:42]

hmm.. I think generally removeAll() on the empty set would result in success.

[robliao, 2014/03/20 19:01:05]

There are two philosophies to take here:
1) We treat this as a success indicator. This means it returns true.
2) We treat this as an indicator work was done. This means it return false.

I'm not sure which is more useful, so let's go with consistency for APIs that
clear everything. chrome.storage.clear uses success. The other ones are
non-obvious at first glance.

On 2014/03/20 18:53:42, Matt Perry wrote:

[Matt Perry, 2014/03/20 21:07:00]

Yeah, #1 makes more sense IMO. I think it should only return false if
num_alarms_remaining != 0.

On 2014/03/20 19:01:05, robliao wrote:

[robliao, 2014/03/20 21:20:30]

We do a hard CHECK if that happens, so we should be good to go for checkin as
is.

If there are no other objections, I'll commit this later today.
On 2014/03/20 21:07:00, Matt Perry wrote:

   SendResponse(true);
 }