Updates:
        Cc: ch...@chromium.org

Comment #2 on issue 29899 by t...@chromium.org: Add mime type to SyncNode interface
http://code.google.com/p/chromium/issues/detail?id=29899

So, if we don't have mime-type in the proto (as issue 29836 now implies), we need to figure out what to do here. I guess we can rely on SyncEntity's specifics()-
HasExtension(sync_pb::bookmark) to tell us that the entity is a bookmark. We could
use this to set the IS_BOOKMARK_OBJECT bit and extend that for all types, but my gut tells me it will be more convenient to use a simple 'ModelType' enum in syncable and a member of this type on the EntryKernel, so we can just GetType rather than ask for each individual type. We could set this by using the HasExtension logic. Thoughts?

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
-- 
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs

Reply via email to