Clean up MenuItemLegacyBarAction

[Cheesebaron]

MenuItemLegacyBarAction has both Context and Activity as arguments, is it really necessary?

[jamesmontemagno]

Most likely not, I think just activity, whichever i use.

[jamesmontemagno]

See my latest commit: jamesmontemagno@7993235

I don't really 100% love how I implemented the "MenuItem" originally, perhaps I need to re-imagine it.....

[Cheesebaron]

What we could do is like ActionBar Sherlock does it. They provide their own MenuItem and their own inflater for it.

[jamesmontemagno]

I believe this is all fixed up basically now, i think it is fine.

[Cheesebaron]

My biggest gripe is, when changing the library to API 14, we will need to implement a bunch of stuff in our MenuItem implementation which is API 14 only, which we don't even use, and have to wrap that in #ANDROID_14 pre-compiler tags, which is suuuuper ugly, and VS thinks there is an error. Will have a look at it this weekend though, I might come up with a nice solution.