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

Side by Side Diff: third_party/WebKit/Source/web/WebLocalFrameImpl.cpp

Issue 2939163003: Removing unused HasHorizontal/VerticalScrollbar methods from WebFrame. (Closed)
Patch Set: Just removing the methods altogether also seems to works... Created 3 years, 6 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) 2009 Google Inc. All rights reserved. 2 * Copyright (C) 2009 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 596 matching lines...) Expand 10 before | Expand all | Expand 10 after
607 return view->VisibleWidth() > 0 && view->VisibleHeight() > 0; 607 return view->VisibleWidth() > 0 && view->VisibleHeight() > 0;
608 return false; 608 return false;
609 } 609 }
610 610
611 WebRect WebLocalFrameImpl::VisibleContentRect() const { 611 WebRect WebLocalFrameImpl::VisibleContentRect() const {
612 if (LocalFrameView* view = GetFrameView()) 612 if (LocalFrameView* view = GetFrameView())
613 return view->VisibleContentRect(); 613 return view->VisibleContentRect();
614 return WebRect(); 614 return WebRect();
615 } 615 }
616 616
617 bool WebLocalFrameImpl::HasHorizontalScrollbar() const {
618 return GetFrame() && GetFrame()->View() &&
619 GetFrame()->View()->HorizontalScrollbar();
620 }
621
622 bool WebLocalFrameImpl::HasVerticalScrollbar() const {
623 return GetFrame() && GetFrame()->View() &&
624 GetFrame()->View()->VerticalScrollbar();
625 }
626
627 WebView* WebLocalFrameImpl::View() const { 617 WebView* WebLocalFrameImpl::View() const {
628 return ViewImpl(); 618 return ViewImpl();
629 } 619 }
630 620
631 WebDocument WebLocalFrameImpl::GetDocument() const { 621 WebDocument WebLocalFrameImpl::GetDocument() const {
632 if (!GetFrame() || !GetFrame()->GetDocument()) 622 if (!GetFrame() || !GetFrame()->GetDocument())
633 return WebDocument(); 623 return WebDocument();
634 return WebDocument(GetFrame()->GetDocument()); 624 return WebDocument(GetFrame()->GetDocument());
635 } 625 }
636 626
(...skipping 1903 matching lines...) Expand 10 before | Expand all | Expand 10 after
2540 WebTextCheckClient* text_check_client) { 2530 WebTextCheckClient* text_check_client) {
2541 text_check_client_ = text_check_client; 2531 text_check_client_ = text_check_client;
2542 } 2532 }
2543 2533
2544 void WebLocalFrameImpl::SetSpellCheckPanelHostClient( 2534 void WebLocalFrameImpl::SetSpellCheckPanelHostClient(
2545 WebSpellCheckPanelHostClient* spell_check_panel_host_client) { 2535 WebSpellCheckPanelHostClient* spell_check_panel_host_client) {
2546 spell_check_panel_host_client_ = spell_check_panel_host_client; 2536 spell_check_panel_host_client_ = spell_check_panel_host_client;
2547 } 2537 }
2548 2538
2549 } // namespace blink 2539 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/web/WebLocalFrameImpl.h ('k') | third_party/WebKit/public/web/WebFrame.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698