On Thu, Apr 15, 2010 at 11:00 PM, Anne van Kesteren <ann...@opera.com>wrote:
> On Wed, 14 Apr 2010 01:13:46 +0900, Mike Belshe <mbel...@google.com> > wrote: > >> // Set the load priority for this request. >> void setPriority(unsigned short priority); >> > > Any reason this is not an attribute named priority? > That is fine with me. > Other than that I wonder if we should maybe simply use string values to > make it easier to extend this if we ever need to. > The thing I don't like about that is the error condition of what happens when someone sets it to "HIHG" (misspelled). Thanks, Mike > > > -- > Anne van Kesteren > http://annevankesteren.nl/ >