Factor out common Element struct from BlobData and ResourceRequestBody

Factor out common Element struct from BlobData and ResourceRequestBody

BUG=110119
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=152987

[kinuko, 2012-08-21 14:30:01]

Add webkit_common::DataElement to replace/merge BlobData::Item and
ResourceRequestBody::Element. The change itself is straightforward, except that
it's adding a new directory webkit/common for webkit-level common basic class. 
Could you take a look?  Thanks!

[jam, 2012-08-21 16:04:01]

chrome&content lgtm

[willchan no longer on Chromium, 2012-08-21 18:13:56]

Is this an FYI for me or is there anything in particular I should look at? The
idea SGTM.

[michaeln, 2012-08-21 19:29:37]

LGTM++

[tony, 2012-08-21 20:48:20]

I defer to darin the introduction of webkit/common.  It's a bit weird since
webkit/common doesn't have the same meaning as chrome/common or content/common,
but I don't have a better naming suggestion.

[kinuko, 2012-08-22 03:44:41]

willchan: it's just FYI, I wanted to let you know I'm deprecating more code
originally from net::UploadData :)

tony: webkit/base maybe?

I'll wait for Darin's comment.

[michaeln1, 2012-08-22 19:01:42]

On 2012/08/22 03:44:41, kinuko wrote:
> willchan: it's just FYI, I wanted to let you know I'm deprecating more code
> originally from net::UploadData :)
> 
> tony: webkit/base maybe?

webkit/base is probably better than webkit/common since it doesn't conflict with
the semantics of chrome/common and the like

[tony, 2012-08-22 20:52:14]

On 2012/08/22 19:01:42, michaeln1 wrote:
> On 2012/08/22 03:44:41, kinuko wrote:
> > willchan: it's just FYI, I wanted to let you know I'm deprecating more code
> > originally from net::UploadData :)
> > 
> > tony: webkit/base maybe?
> 
> webkit/base is probably better than webkit/common since it doesn't conflict
with
> the semantics of chrome/common and the like

Darin might be on vacation, so let's just go with with webkit/base for now.  We
can always rename it later.  LGTM after renaming to webkit/base.

[commit-bot: I haz the power, 2012-08-23 09:07:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinuko@chromium.org/10827414/18001

[commit-bot: I haz the power, 2012-08-23 14:32:45]

Change committed as 152987

[darin (slow to review), 2012-08-27 18:16:57]

LGTM2  ... I had been planning on creating webkit/base for other (similar)
reasons.
-Darin

On Thu, Aug 23, 2012 at 7:32 AM, <commit-bot@chromium.org> wrote:

> Change committed as 152987
>
>
https://chromiumcodereview.**appspot.com/10827414/<https://chromiumcodereview...
>