You'd really write code like that? I dislike having computers shout at me. All caps macros may be traditional in C programming, but so are missing declarations defaulting to "int", lack of comments, and "if (x = 1)" bugs.
Sometimes it is useful to write individual macros like these (especially when you are not sure of the polarities of signals before testing), but I think it is a help to consistency if you use parenthesis on all macros:
#define ledOn() cbit(ledBit, PORTB)
That way you can more easily change things in the future, such as changing a macro definition to an inline function call (I mostly prefer "static inline" functions to macros, if the compiler handles them well).
And put parenthesis around "bit" in your bit macros, for safety (that's one of the reasons I prefer static inline functions).