Support dispositon attribute.

Support dispositon attribute.

R=jhawkins@chromium.org
BUG=None
TEST=None


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

[James Hawkins, 2011-10-28 22:25:36]

Are you adding the webdb registration code in a followup CL?

http://codereview.chromium.org/8343070/diff/1/content/browser/tab_contents/ta...
File content/browser/tab_contents/tab_contents.cc (right):

http://codereview.chromium.org/8343070/diff/1/content/browser/tab_contents/ta...
content/browser/tab_contents/tab_contents.cc:282: OnRegisterIntentHandler)
These no longer match the implementation order.

[Greg Billock, 2011-10-28 23:55:21]

webdb registration is already done; it just wasn't hooked up to anything. I have
the webkit patch already merged into my client for the attribute (still need to
get it into the github repo).

http://codereview.chromium.org/8343070/diff/1/content/browser/tab_contents/ta...
File content/browser/tab_contents/tab_contents.cc (right):

http://codereview.chromium.org/8343070/diff/1/content/browser/tab_contents/ta...
content/browser/tab_contents/tab_contents.cc:282: OnRegisterIntentHandler)
On 2011/10/28 22:25:37, James Hawkins wrote:
> These no longer match the implementation order.

Yeah. The impl matched header order. The organization is a bit... opaque. :-/ I
moved them around to be in front of the ViewHostMsg everywhere.

[James Hawkins, 2011-10-29 22:27:43]

lgtm

[Greg Billock, 2011-10-31 20:44:56]

On 2011/10/29 22:27:43, James Hawkins wrote:
> lgtm

John, can you look at this for content/*. Thanks!

[jam, 2011-10-31 21:24:25]

lgtm

[commit-bot: I haz the power, 2011-10-31 21:36:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gbillock@chromium.org/8343070/31

[commit-bot: I haz the power, 2011-10-31 21:36:09]

Can't apply patch for file content/browser/tab_contents/tab_contents.cc.
While running patch -p1 --forward --force;
patching file content/browser/tab_contents/tab_contents.cc
Hunk #5 FAILED at 2008.
1 out of 5 hunks FAILED -- saving rejects to file
content/browser/tab_contents/tab_contents.cc.rej

[commit-bot: I haz the power, 2011-10-31 22:14:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gbillock@chromium.org/8343070/8001

[commit-bot: I haz the power, 2011-10-31 22:14:10]

Can't process patch for file content/common/view_messages.h.
File's status is None, patchset upload is incomplete.

[commit-bot: I haz the power, 2011-10-31 22:44:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gbillock@chromium.org/8343070/4002

[commit-bot: I haz the power, 2011-10-31 23:56:31]

Change committed as 108042