Reland of place string::find(prefix) == 0 pattern with base::StartsWith().

Reland of place string::find(prefix) == 0 pattern with base::StartsWith(). (patchset #1 id:1 of https://codereview.chromium.org/2131933002/ )

Reason for revert:
Reverting this revert since https://codereview.chromium.org/2135653002/

seems to fix the issue

Original issue's description:
> Revert of Replace string::find(prefix) == 0 pattern with base::StartsWith(). (patchset #2 id:20001 of https://codereview.chromium.org/2121513002/ )
>
> Reason for revert:
> Bisect believes that this CL caused the Win builder to break.
>
> https://build.chromium.org/p/chromium/builders/Win%20x64/builds/2267
> https://build.chromium.org/p/chromium/builders/Win/builds/45048
>
> For context see:
> https://bugs.chromium.org/p/chromium/issues/detail?id=626539
>
> Original issue's description:
> > Replace string::find(prefix) == 0 pattern with base::StartsWith().
> >
> > base::StartsWith() should be O(|prefix|) whereas string::find() is
> >   pretty bad, O(|string| * |prefix|).
> >
> > TBR=jam@chromium.org
> > BUG=None
> > Test=None, internal cleanup.
> > CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win10_chromium_x64_rel_ng
> >
> > Committed: https://crrev.com/53bfbe932191085b5d69f824a9923bccc26d58dc
> > Cr-Commit-Position: refs/heads/master@{#404307}
>
> TBR=thestig@chromium.org,lazyboy@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=None
>
> Committed: https://crrev.com/f635836294b0221de8531e3ab3720d7fc18e65cd
> Cr-Commit-Position: refs/heads/master@{#404341}

TBR=thestig@chromium.org,lazyboy@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=None

Committed: https://crrev.com/74e10b22e14d23d306351e94e3db01ca8f04ee8c
Cr-Commit-Position: refs/heads/master@{#404342}

[markusheintz_, 2016-07-08 13:17:51]

The CQ bit was checked by markusheintz@chromium.org

[markusheintz_, 2016-07-08 13:17:52]

Created Reland of place string::find(prefix) == 0 pattern with
base::StartsWith().

[commit-bot: I haz the power, 2016-07-08 13:18:41]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-08 13:19:59]

Description was changed from

==========
Reland of place string::find(prefix) == 0 pattern with base::StartsWith().
(patchset #1 id:1 of https://codereview.chromium.org/2131933002/ )

Reason for revert:
Reverting this revert since https://codereview.chromium.org/2135653002/

seems to fix the issue

Original issue's description:
> Revert of Replace string::find(prefix) == 0 pattern with base::StartsWith().
(patchset #2 id:20001 of https://codereview.chromium.org/2121513002/ )
> 
> Reason for revert:
> Bisect believes that this CL caused the Win builder to break.
> 
> https://build.chromium.org/p/chromium/builders/Win%20x64/builds/2267
> https://build.chromium.org/p/chromium/builders/Win/builds/45048
> 
> For context see:
> https://bugs.chromium.org/p/chromium/issues/detail?id=626539
> 
> Original issue's description:
> > Replace string::find(prefix) == 0 pattern with base::StartsWith().
> >
> > base::StartsWith() should be O(|prefix|) whereas string::find() is
> >   pretty bad, O(|string| * |prefix|).
> >
> > TBR=jam@chromium.org
> > BUG=None
> > Test=None, internal cleanup.
> > CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win10_chromium_x64_rel_ng
> >
> > Committed: https://crrev.com/53bfbe932191085b5d69f824a9923bccc26d58dc
> > Cr-Commit-Position: refs/heads/master@{#404307}
> 
> TBR=thestig@chromium.org,lazyboy@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=None
> 
> Committed: https://crrev.com/f635836294b0221de8531e3ab3720d7fc18e65cd
> Cr-Commit-Position: refs/heads/master@{#404341}

TBR=thestig@chromium.org,lazyboy@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=None
==========

to

==========
Reland of place string::find(prefix) == 0 pattern with base::StartsWith().
(patchset #1 id:1 of https://codereview.chromium.org/2131933002/ )

Reason for revert:
Reverting this revert since https://codereview.chromium.org/2135653002/

seems to fix the issue

Original issue's description:
> Revert of Replace string::find(prefix) == 0 pattern with base::StartsWith().
(patchset #2 id:20001 of https://codereview.chromium.org/2121513002/ )
> 
> Reason for revert:
> Bisect believes that this CL caused the Win builder to break.
> 
> https://build.chromium.org/p/chromium/builders/Win%20x64/builds/2267
> https://build.chromium.org/p/chromium/builders/Win/builds/45048
> 
> For context see:
> https://bugs.chromium.org/p/chromium/issues/detail?id=626539
> 
> Original issue's description:
> > Replace string::find(prefix) == 0 pattern with base::StartsWith().
> >
> > base::StartsWith() should be O(|prefix|) whereas string::find() is
> >   pretty bad, O(|string| * |prefix|).
> >
> > TBR=jam@chromium.org
> > BUG=None
> > Test=None, internal cleanup.
> > CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win10_chromium_x64_rel_ng
> >
> > Committed: https://crrev.com/53bfbe932191085b5d69f824a9923bccc26d58dc
> > Cr-Commit-Position: refs/heads/master@{#404307}
> 
> TBR=thestig@chromium.org,lazyboy@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=None
> 
> Committed: https://crrev.com/f635836294b0221de8531e3ab3720d7fc18e65cd
> Cr-Commit-Position: refs/heads/master@{#404341}

TBR=thestig@chromium.org,lazyboy@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=None
==========

[commit-bot: I haz the power, 2016-07-08 13:20:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-08 13:20:21]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-08 13:21:53]

Description was changed from

==========
Reland of place string::find(prefix) == 0 pattern with base::StartsWith().
(patchset #1 id:1 of https://codereview.chromium.org/2131933002/ )

Reason for revert:
Reverting this revert since https://codereview.chromium.org/2135653002/

seems to fix the issue

Original issue's description:
> Revert of Replace string::find(prefix) == 0 pattern with base::StartsWith().
(patchset #2 id:20001 of https://codereview.chromium.org/2121513002/ )
> 
> Reason for revert:
> Bisect believes that this CL caused the Win builder to break.
> 
> https://build.chromium.org/p/chromium/builders/Win%20x64/builds/2267
> https://build.chromium.org/p/chromium/builders/Win/builds/45048
> 
> For context see:
> https://bugs.chromium.org/p/chromium/issues/detail?id=626539
> 
> Original issue's description:
> > Replace string::find(prefix) == 0 pattern with base::StartsWith().
> >
> > base::StartsWith() should be O(|prefix|) whereas string::find() is
> >   pretty bad, O(|string| * |prefix|).
> >
> > TBR=jam@chromium.org
> > BUG=None
> > Test=None, internal cleanup.
> > CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win10_chromium_x64_rel_ng
> >
> > Committed: https://crrev.com/53bfbe932191085b5d69f824a9923bccc26d58dc
> > Cr-Commit-Position: refs/heads/master@{#404307}
> 
> TBR=thestig@chromium.org,lazyboy@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=None
> 
> Committed: https://crrev.com/f635836294b0221de8531e3ab3720d7fc18e65cd
> Cr-Commit-Position: refs/heads/master@{#404341}

TBR=thestig@chromium.org,lazyboy@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=None
==========

to

==========
Reland of place string::find(prefix) == 0 pattern with base::StartsWith().
(patchset #1 id:1 of https://codereview.chromium.org/2131933002/ )

Reason for revert:
Reverting this revert since https://codereview.chromium.org/2135653002/

seems to fix the issue

Original issue's description:
> Revert of Replace string::find(prefix) == 0 pattern with base::StartsWith().
(patchset #2 id:20001 of https://codereview.chromium.org/2121513002/ )
>
> Reason for revert:
> Bisect believes that this CL caused the Win builder to break.
>
> https://build.chromium.org/p/chromium/builders/Win%20x64/builds/2267
> https://build.chromium.org/p/chromium/builders/Win/builds/45048
>
> For context see:
> https://bugs.chromium.org/p/chromium/issues/detail?id=626539
>
> Original issue's description:
> > Replace string::find(prefix) == 0 pattern with base::StartsWith().
> >
> > base::StartsWith() should be O(|prefix|) whereas string::find() is
> >   pretty bad, O(|string| * |prefix|).
> >
> > TBR=jam@chromium.org
> > BUG=None
> > Test=None, internal cleanup.
> > CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win10_chromium_x64_rel_ng
> >
> > Committed: https://crrev.com/53bfbe932191085b5d69f824a9923bccc26d58dc
> > Cr-Commit-Position: refs/heads/master@{#404307}
>
> TBR=thestig@chromium.org,lazyboy@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=None
>
> Committed: https://crrev.com/f635836294b0221de8531e3ab3720d7fc18e65cd
> Cr-Commit-Position: refs/heads/master@{#404341}

TBR=thestig@chromium.org,lazyboy@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=None

Committed: https://crrev.com/74e10b22e14d23d306351e94e3db01ca8f04ee8c
Cr-Commit-Position: refs/heads/master@{#404342}
==========

[commit-bot: I haz the power, 2016-07-08 13:21:54]

Patchset 1 (id:??) landed as
https://crrev.com/74e10b22e14d23d306351e94e3db01ca8f04ee8c
Cr-Commit-Position: refs/heads/master@{#404342}