Get weborigin to build within blink_platform target.

Get weborigin to build within blink_platform target.

It does not build entirely yet, but get us closer.

../../third_party/WebKit/Source/platform/weborigin/OriginAccessEntry.cpp:
In constructor 'WebCore::OriginAccessEntry::OriginAccessEntry(const
                WTF::String&, const WTF::String&,
                WebCore::OriginAccessEntry::SubdomainSetting,
                WebCore::OriginAccessEntry::IPAddressSetting)':
../../third_party/WebKit/Source/platform/weborigin/OriginAccessEntry.cpp:55:86:
error: no matching function for call to
'blink::WebPublicSuffixList::getPublicSuffixLength(WTF::String&)'
../../third_party/WebKit/Source/platform/weborigin/OriginAccessEntry.cpp:55:86:
note: candidate is:
../../third_party/WebKit/public/platform/WebPublicSuffixList.h:40:20:
note: virtual size_t
blink::WebPublicSuffixList::getPublicSuffixLength(const
                blink::WebString&)
../../third_party/WebKit/public/platform/WebPublicSuffixList.h:40:20:
note:   no known conversion for argument 1 from 'WTF::String' to 'const
blink::WebString&'

BUG=328717
TEST=gn, ninja -C out/Default blink_platform
R=brettw@chromium.org
TBR=darin

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

[brettw, 2014-01-08 05:29:59]

lgtm

https://codereview.chromium.org/119723003/diff/70001/tools/gn/secondary/third...
File tools/gn/secondary/third_party/WebKit/Source/platform/BUILD.gn (right):

https://codereview.chromium.org/119723003/diff/70001/tools/gn/secondary/third...
tools/gn/secondary/third_party/WebKit/Source/platform/BUILD.gn:5:
component("blink_platform") {
Can you call this "platform" so people can refer to it more conveniently, then
do in the body
  output_name = "blink_platform" so the output file (which goes in a directory
with all the other libraries) is more unique.

[tfarina, 2014-01-09 00:59:43]

https://codereview.chromium.org/119723003/diff/70001/tools/gn/secondary/third...
File tools/gn/secondary/third_party/WebKit/Source/platform/BUILD.gn (right):

https://codereview.chromium.org/119723003/diff/70001/tools/gn/secondary/third...
tools/gn/secondary/third_party/WebKit/Source/platform/BUILD.gn:5:
component("blink_platform") {
On 2014/01/08 05:29:59, brettw wrote:
> Can you call this "platform" so people can refer to it more conveniently, then
> do in the body
>   output_name = "blink_platform" so the output file (which goes in a directory
> with all the other libraries) is more unique.

Done.

[commit-bot: I haz the power, 2014-01-09 01:06:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tfarina@chromium.org/119723003/170001

[commit-bot: I haz the power, 2014-01-09 01:47:35]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[tfarina, 2014-01-09 02:45:29]

TBRing Darin for that.

Looks like src/OWNERS needs to be updated!

[commit-bot: I haz the power, 2014-01-09 02:48:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tfarina@chromium.org/119723003/170001

[commit-bot: I haz the power, 2014-01-09 03:47:00]

Retried try job too often on ios_dbg_simulator for step(s) ui_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=ios_dbg_si...

[commit-bot: I haz the power, 2014-01-09 12:50:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tfarina@chromium.org/119723003/170001

[commit-bot: I haz the power, 2014-01-09 14:08:54]

Change committed as 243886