Don't set MODE_DEFAULT when adding HPKP header.

Don't set MODE_DEFAULT when adding HPKP header.

Leave that to the default value, or whatever value an HSTS header set.

TEST=net_unittests; after visiting https://www.mv-rechberghausen.de, query
that domain in chrome://net-internal/#hsts and expect pins AND mode STRICT.

BUG=226068
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=193443

[agl, 2013-04-03 21:46:35]

lgtm

[Ryan Sleevi, 2013-04-03 21:52:47]

Can you please add a new unittest for this?

Should be as simple as adding some mock headers for a domain like the test.

[commit-bot: I haz the power, 2013-04-04 17:56:18]

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

[commit-bot: I haz the power, 2013-04-04 20:05:32]

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

[commit-bot: I haz the power, 2013-04-04 20:07:25]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac&number...

[commit-bot: I haz the power, 2013-04-08 22:26:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/palmer@chromium.org/13483007/44002

[commit-bot: I haz the power, 2013-04-09 01:49:49]

Retried try job too often on win_rel for step(s) chrome_frame_net_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-04-09 19:48:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/palmer@chromium.org/13483007/62001

[commit-bot: I haz the power, 2013-04-09 19:48:23]

Presubmit check for 13483007-62001 failed and returned exit status 1.

INFO:root:Found 9 file(s).

Running presubmit commit checks ...
Running /b/commit-queue/workdir/chromium/PRESUBMIT.py
Running /b/commit-queue/workdir/chromium/net/PRESUBMIT.py

** Presubmit ERRORS **
Missing LGTM from an OWNER for these files:
    chrome_frame/test/net/fake_external_tab.cc

Presubmit checks took 1.1s to calculate.

[palmer, 2013-04-09 20:07:01]

Ananta, can I please get an OWNERS LGTM for the Chrome Frame file? Thanks.

[ananta, 2013-04-10 17:55:19]

lgtm

[commit-bot: I haz the power, 2013-04-10 17:58:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/palmer@chromium.org/13483007/62001

[commit-bot: I haz the power, 2013-04-10 20:24:38]

Change committed as 193443