[DevTools] Remove V8ContentSearchUtil.

[DevTools] Remove V8ContentSearchUtil.

BUG=580337
TBR=pfeldman

Committed: https://crrev.com/42ab3ad19bbbc798996b3226f8b1be692debae6d
Cr-Commit-Position: refs/heads/master@{#408567}

[dgozman, 2016-07-27 01:10:13]

dgozman@chromium.org changed reviewers:
+ kozyatinskiy@chromium.org

[dgozman, 2016-07-27 01:10:14]

Take a look please.

[kozy, 2016-07-27 01:17:05]

lgtm

https://codereview.chromium.org/2182423002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/v8_inspector/V8InspectorSessionImpl.h
(right):

https://codereview.chromium.org/2182423002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/v8_inspector/V8InspectorSessionImpl.h:67:
std::unique_ptr<protocol::Array<protocol::Debugger::API::SearchMatch>>
searchInTextByLines(const uint16_t* text, size_t textLength, const uint16_t*
query, size_t queryLength, const bool caseSensitive, const bool isRegex)
override;
"const bool" -> "bool"

[dgozman, 2016-07-27 01:19:12]

https://codereview.chromium.org/2182423002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/v8_inspector/V8InspectorSessionImpl.h
(right):

https://codereview.chromium.org/2182423002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/v8_inspector/V8InspectorSessionImpl.h:67:
std::unique_ptr<protocol::Array<protocol::Debugger::API::SearchMatch>>
searchInTextByLines(const uint16_t* text, size_t textLength, const uint16_t*
query, size_t queryLength, const bool caseSensitive, const bool isRegex)
override;
On 2016/07/27 01:17:05, kozyatinskiy wrote:
> "const bool" -> "bool"

Done.

[dgozman, 2016-07-27 01:19:16]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-07-27 01:19:16]

The patchset sent to the CQ was uploaded after l-g-t-m from
kozyatinskiy@chromium.org
Link to the patchset: https://codereview.chromium.org/2182423002/#ps60001
(title: "const bool")

[dgozman, 2016-07-27 01:19:20]

The CQ bit was unchecked by dgozman@chromium.org

[dgozman, 2016-07-27 16:06:39]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2016-07-27 16:06:56]

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-27 16:12:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-27 16:12:33]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[dgozman, 2016-07-27 16:25:59]

Description was changed from

==========
[DevTools] Migrate searchInTextByLines to uint16_t* from String16.

BUG=580337
==========

to

==========
[DevTools] Migrate searchInTextByLines to uint16_t* from String16.

BUG=580337
TBR=pfeldman
==========

[dgozman, 2016-07-27 16:26:33]

dgozman@chromium.org changed reviewers:
+ pfeldman@chromium.org

[dgozman, 2016-07-27 16:26:34]

TBR'ing pfeldman@ for gypi.

[dgozman, 2016-07-27 16:26:39]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2016-07-27 16:26:56]

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-27 16:37:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-27 16:37:14]

Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[dgozman, 2016-07-29 01:28:05]

Description was changed from

==========
[DevTools] Migrate searchInTextByLines to uint16_t* from String16.

BUG=580337
TBR=pfeldman
==========

to

==========
[DevTools] Remove V8ContentSearchUtil.

BUG=580337
TBR=pfeldman
==========

[dgozman, 2016-07-29 01:42:25]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-07-29 01:42:25]

The patchset sent to the CQ was uploaded after l-g-t-m from
kozyatinskiy@chromium.org
Link to the patchset: https://codereview.chromium.org/2182423002/#ps80001
(title: "just removal")

[commit-bot: I haz the power, 2016-07-29 01:42:42]

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-29 03:19:57]

Description was changed from

==========
[DevTools] Remove V8ContentSearchUtil.

BUG=580337
TBR=pfeldman
==========

to

==========
[DevTools] Remove V8ContentSearchUtil.

BUG=580337
TBR=pfeldman
==========

[commit-bot: I haz the power, 2016-07-29 03:19:58]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-07-29 03:24:23]

Description was changed from

==========
[DevTools] Remove V8ContentSearchUtil.

BUG=580337
TBR=pfeldman
==========

to

==========
[DevTools] Remove V8ContentSearchUtil.

BUG=580337
TBR=pfeldman

Committed: https://crrev.com/42ab3ad19bbbc798996b3226f8b1be692debae6d
Cr-Commit-Position: refs/heads/master@{#408567}
==========

[commit-bot: I haz the power, 2016-07-29 03:24:24]

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