Apply WebSettings before initializing the main frame.

Apply WebSettings before initializing the main frame.

Some settings need to be applied before initializing the main frame,
since they affect the creation of the initial empty document.

BUG=594383
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/0ff94cfc829f6df331dd71df1eef201e06cca36c
Cr-Commit-Position: refs/heads/master@{#381327}

[dcheng, 2016-03-15 08:30:31]

Description was changed from

==========
Apply WebSettings before initializing the main frame.

BUG=594383
==========

to

==========
Apply WebSettings before initializing the main frame.

Some settings need to be applied before initializing the main frame,
since they affect the creation of the initial empty document.

BUG=594383
==========

[dcheng, 2016-03-15 08:30:31]

dcheng@chromium.org changed reviewers:
+ jochen@chromium.org

[dcheng, 2016-03-15 08:33:08]

Description was changed from

==========
Apply WebSettings before initializing the main frame.

Some settings need to be applied before initializing the main frame,
since they affect the creation of the initial empty document.

BUG=594383
==========

to

==========
Apply WebSettings before initializing the main frame.

Some settings need to be applied before initializing the main frame,
since they affect the creation of the initial empty document.

BUG=594383
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[dcheng, 2016-03-15 08:33:54]

https://codereview.chromium.org/1799423003/diff/20001/content/renderer/render...
File content/renderer/render_view_impl.cc (right):

https://codereview.chromium.org/1799423003/diff/20001/content/renderer/render...
content/renderer/render_view_impl.cc:772: UpdateWebViewWithDeviceScaleFactor();
One might wonder: why are these here?
Answer: this is the configuration that doesn't crash.

Unsatisfying, but this bit of code is essentially code soup. =(

[jochen (gone - plz use gerrit), 2016-03-15 08:36:14]

test? :)

[dcheng, 2016-03-15 09:27:55]

On 2016/03/15 at 08:36:14, jochen wrote:
> test? :)

Done.

[dcheng, 2016-03-15 09:28:25]

On 2016/03/15 at 09:27:55, dcheng wrote:
> On 2016/03/15 at 08:36:14, jochen wrote:
> > test? :)
> 
> Done.

Oops, need to rebuild and upload the right expectations.

[dcheng, 2016-03-15 09:31:06]

On 2016/03/15 at 09:28:25, dcheng wrote:
> On 2016/03/15 at 09:27:55, dcheng wrote:
> > On 2016/03/15 at 08:36:14, jochen wrote:
> > > test? :)
> > 
> > Done.
> 
> Oops, need to rebuild and upload the right expectations.

Expectations fixed.

[jochen (gone - plz use gerrit), 2016-03-15 20:59:51]

lgtm

[dcheng, 2016-03-15 21:06:31]

The CQ bit was checked by dcheng@chromium.org

[commit-bot: I haz the power, 2016-03-15 21:07:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1799423003/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1799423003/80001

[commit-bot: I haz the power, 2016-03-15 21:51:04]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-03-15 21:52:17]

Description was changed from

==========
Apply WebSettings before initializing the main frame.

Some settings need to be applied before initializing the main frame,
since they affect the creation of the initial empty document.

BUG=594383
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Apply WebSettings before initializing the main frame.

Some settings need to be applied before initializing the main frame,
since they affect the creation of the initial empty document.

BUG=594383
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/0ff94cfc829f6df331dd71df1eef201e06cca36c
Cr-Commit-Position: refs/heads/master@{#381327}
==========

[commit-bot: I haz the power, 2016-03-15 21:52:18]

Patchset 5 (id:??) landed as
https://crrev.com/0ff94cfc829f6df331dd71df1eef201e06cca36c
Cr-Commit-Position: refs/heads/master@{#381327}