Remove default args to several PageWidgetDelegate members, try 2.

Remove default args to several PageWidgetDelegate members, try 2.

BUG=411993
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=182737

[dcheng, 2014-09-25 22:57:10]

dcheng@chromium.org changed reviewers:
+ japhet@chromium.org, kenrb@chromium.org

[dcheng, 2014-09-25 22:57:11]

The temporary check that the main frame is a local root is needed to keep at
least three content_browsertests happy:
SitePerProcessBrowserTest.CrossSiteIframe
SitePerProcessAccessibilityBrowserTest.CrossSiteIframeAccessibility
CrossProcessFrameTreeBrowserTest.CreateCrossProcessSubframeProxies

I believe Ken's later patches will eliminate the need for this null check when
we correctly use the local root (?).

[Nate Chapin, 2014-09-25 23:02:44]

lgtm

[dcheng, 2014-09-26 06:24:00]

The CQ bit was checked by dcheng@chromium.org

[commit-bot: I haz the power, 2014-09-26 06:24:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/604463004/20001

[commit-bot: I haz the power, 2014-09-26 06:35:19]

Committed patchset #2 (id:20001) as 182737

[kenrb, 2014-09-26 14:06:20]

On 2014/09/25 22:57:11, dcheng wrote:
> The temporary check that the main frame is a local root is needed to keep at
> least three content_browsertests happy:
> SitePerProcessBrowserTest.CrossSiteIframe
> SitePerProcessAccessibilityBrowserTest.CrossSiteIframeAccessibility
> CrossProcessFrameTreeBrowserTest.CreateCrossProcessSubframeProxies
> 
> I believe Ken's later patches will eliminate the need for this null check when
> we correctly use the local root (?).

Yes.

[dcheng, 2014-09-30 00:04:26]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/598503003/ by dcheng@chromium.org.

The reason for reverting is: Causing crashes on canary..