Support prop defaults being resource identifiers about litho

[pavlospt]

I have started working on this and my progress so far is:

• I have added the ability to add a ResType on a PropDefault annotation.
• I extract it successfully and add it on the PropDefaultModel
• Parsing it correctly and initializing the Prop field with a proper initializer (only using the resolveDimenSizeRes() for now)

My thoughts on the above:

• In order to use the resource resolve method, I tweaked the Component class to extend ResourceResolver and code generation works properly now. Although what this does, is crash on Runtime because it does not have a valid ComponentContext to call super.init(c, c.getResourceCache()).

I do not know if I am on the right path yet, but it seems to me that in order to support this we need to either extend ResourceResolver from Component and provide it with a valid ComponentContext or just initialize the PropDefault marked variable on the Builder where we have a valid ComponentContext.

Let me know what you think.

/cc: @emilsjolander @passy

P.S.: Sorry for pinging here as well, might seem like spamming, but I want to save the time for anyone who would like to start working on this one and we end up having duplicate work.

[emilsjolander]

Could you publish your current progress as a pull request? This way we could make better suggestions based on the code. From the info you provided I think both alternatives would work. I see no reason not to pass the context down to the component.

[pavlospt]

@emilsjolander Ok will try to prepare a PR for them later in the day or tomorrow :)

Edit: Only thought on passing the ComponentContext down to Component, is that it would introduce a breaking change If I am not mistaken.

[pavlospt]

Can be closed!