Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(761)

Issue 132563003: Revert 164780 "Introduce TextFinder class for decoupling WebFram..." (Closed)

Created:
6 years, 11 months ago by Fady Samuel
Modified:
6 years, 11 months ago
Reviewers:
Andrey Kraynov
CC:
blink-reviews
Visibility:
Public.

Description

Revert 164780 "Introduce TextFinder class for decoupling WebFram..." > Introduce TextFinder class for decoupling WebFrameImpl and text finder. > > It will be lazily initialized when user activates find on page functionality. > > BUG= > > Review URL: https://codereview.chromium.org/69923006 TBR=iceman@yandex-team.ru Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=164801

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2953 lines, -269 lines) Patch
M trunk/Source/web/TextFinder.h View 7 chunks +309 lines, -53 lines 0 comments Download
M trunk/Source/web/TextFinder.cpp View 25 chunks +1828 lines, -148 lines 0 comments Download
M trunk/Source/web/WebFrameImpl.h View 4 chunks +169 lines, -9 lines 0 comments Download
M trunk/Source/web/WebFrameImpl.cpp View 13 chunks +647 lines, -57 lines 0 comments Download
M trunk/Source/web/web.gypi View 1 chunk +0 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Fady Samuel
6 years, 11 months ago (2014-01-09 20:44:19 UTC) #1
Fady Samuel
6 years, 11 months ago (2014-01-09 20:45:01 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r164801.

Powered by Google App Engine
This is Rietveld 408576698