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

Side by Side Diff: third_party/WebKit/Source/web/WebViewImpl.h

Issue 2674253004: [refactor] Remove WebWidget::applyReplacementRange (Closed)
Patch Set: Rebased Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2010 Google Inc. All rights reserved. 2 * Copyright (C) 2010 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 511 matching lines...) Expand 10 before | Expand all | Expand 10 after
522 void resizeViewWhileAnchored(float browserControlsHeight, 522 void resizeViewWhileAnchored(float browserControlsHeight,
523 bool browserControlsShrinkLayout); 523 bool browserControlsShrinkLayout);
524 524
525 // Overrides the compositor visibility. See the description of 525 // Overrides the compositor visibility. See the description of
526 // m_overrideCompositorVisibility for more details. 526 // m_overrideCompositorVisibility for more details.
527 void setCompositorVisibility(bool); 527 void setCompositorVisibility(bool);
528 528
529 // TODO(lfg): Remove once WebViewFrameWidget is deleted. 529 // TODO(lfg): Remove once WebViewFrameWidget is deleted.
530 void scheduleAnimationForWidget(); 530 void scheduleAnimationForWidget();
531 bool getCompositionCharacterBounds(WebVector<WebRect>&) override; 531 bool getCompositionCharacterBounds(WebVector<WebRect>&) override;
532 void applyReplacementRange(const WebRange&) override;
533 532
534 void updateBaseBackgroundColor(); 533 void updateBaseBackgroundColor();
535 534
536 friend class WebView; // So WebView::Create can call our constructor 535 friend class WebView; // So WebView::Create can call our constructor
537 friend class WebViewFrameWidget; 536 friend class WebViewFrameWidget;
538 friend class WTF::RefCounted<WebViewImpl>; 537 friend class WTF::RefCounted<WebViewImpl>;
539 friend void setCurrentInputEventForTest(const WebInputEvent*); 538 friend void setCurrentInputEventForTest(const WebInputEvent*);
540 539
541 explicit WebViewImpl(WebViewClient*, WebPageVisibilityState); 540 explicit WebViewImpl(WebViewClient*, WebPageVisibilityState);
542 ~WebViewImpl() override; 541 ~WebViewImpl() override;
(...skipping 198 matching lines...) Expand 10 before | Expand all | Expand 10 after
741 Persistent<ResizeViewportAnchor> m_resizeViewportAnchor; 740 Persistent<ResizeViewportAnchor> m_resizeViewportAnchor;
742 }; 741 };
743 742
744 // We have no ways to check if the specified WebView is an instance of 743 // We have no ways to check if the specified WebView is an instance of
745 // WebViewImpl because WebViewImpl is the only implementation of WebView. 744 // WebViewImpl because WebViewImpl is the only implementation of WebView.
746 DEFINE_TYPE_CASTS(WebViewImpl, WebView, webView, true, true); 745 DEFINE_TYPE_CASTS(WebViewImpl, WebView, webView, true, true);
747 746
748 } // namespace blink 747 } // namespace blink
749 748
750 #endif 749 #endif
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/web/WebViewFrameWidget.cpp ('k') | third_party/WebKit/Source/web/WebViewImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698