Add RequestUpdate to NetworkSmsHandler

Add RequestUpdate to NetworkSmsHandler


BUG=124269
TEST=None yet, except that tests pass; will be used for further testing.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=137507

[stevenjb, 2012-05-16 20:53:09]

Small addition for testing the UI with the stub (so that no fake SMS messages
will be sent until the status tray is opened)

[satorux1, 2012-05-16 20:59:10]

LGTM

http://codereview.chromium.org/10391170/diff/3007/chromeos/network/network_sm...
File chromeos/network/network_sms_handler.cc (right):

http://codereview.chromium.org/10391170/diff/3007/chromeos/network/network_sm...
chromeos/network/network_sms_handler.cc:164: (*iter)->RequestUpdate();
nit: add {} ? If I understand right, we omit {} only if the conditional and the
body are both one line.

[stevenjb, 2012-05-16 21:12:13]

http://codereview.chromium.org/10391170/diff/3007/chromeos/network/network_sm...
File chromeos/network/network_sms_handler.cc (right):

http://codereview.chromium.org/10391170/diff/3007/chromeos/network/network_sm...
chromeos/network/network_sms_handler.cc:164: (*iter)->RequestUpdate();
On 2012/05/16 20:59:10, satorux1 wrote:
> nit: add {} ? If I understand right, we omit {} only if the conditional and
the
> body are both one line.

Done.