Don't use method cache when getting clinit
Don't use method cache when getting clinit

Summary:
Previously we consulted the method cache when fetching the
clinit from a DexClass. As a result, one could not observe the effects
of removing a clinit when calling `clazz->get_clinit()`. In code, that
looks like:

        auto clinit = clazz->get_clinit()
        clazz->remove_method(clinit)
        // This is false:
        nullptr == clazz->get_clinit()

Reviewed By: bertmaher

Differential Revision: D4522529

fbshipit-source-id: 4ecd42bae9653e6e06ccbf6a60f186dc76117f40

facebook/redex,facebook/redex,facebook/redex,facebook/redex,facebook/redex,facebook/redex,facebook/redex,facebook/redex,facebook/redex