add mole collapse/expand events. convert mappy to use this.

add mole collapse/expand events.  convert mappy to use this.
BUG=15494
TEST=browser_tests.exe --gtest_filter=ExtensionApiTest.Toolstrip

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=25976

[Matt Perry, 2009-09-10 23:02:50]

LGTM

http://codereview.chromium.org/203023/diff/4001/5002
File chrome/browser/extensions/extension_toolstrip_api.cc (right):

http://codereview.chromium.org/203023/diff/4001/5002#newcode166
Line 166: if (!url.is_empty())
should we set the .url property to be "" rather than undefined in this case?

[Erik does not do reviews, 2009-09-10 23:07:18]

On Thu, Sep 10, 2009 at 4:02 PM,  <mpcomplete@chromium.org> wrote:
> LGTM
>
>
> http://codereview.chromium.org/203023/diff/4001/5002
> File chrome/browser/extensions/extension_toolstrip_api.cc (right):
>
> http://codereview.chromium.org/203023/diff/4001/5002#newcode166
> Line 166: if (!url.is_empty())
> should we set the .url property to be "" rather than undefined in this
> case?

The style of our other "update-style" APIs is to leave properties out
if they haven't changed.

Erik