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

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

Issue 2905283003: Remove a bunch of dead code around WindowFeatures (Closed)
Patch Set: Use WebWindowFeatures everywhere 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) 2011, 2012 Google Inc. All rights reserved. 2 * Copyright (C) 2011, 2012 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 3555 matching lines...) Expand 10 before | Expand all | Expand 10 after
3566 return GetPage() ? GetPage()->GetFocusController().IsActive() : false; 3566 return GetPage() ? GetPage()->GetFocusController().IsActive() : false;
3567 } 3567 }
3568 3568
3569 void WebViewImpl::SetDomainRelaxationForbidden(bool forbidden, 3569 void WebViewImpl::SetDomainRelaxationForbidden(bool forbidden,
3570 const WebString& scheme) { 3570 const WebString& scheme) {
3571 SchemeRegistry::SetDomainRelaxationForbiddenForURLScheme(forbidden, 3571 SchemeRegistry::SetDomainRelaxationForbiddenForURLScheme(forbidden,
3572 String(scheme)); 3572 String(scheme));
3573 } 3573 }
3574 3574
3575 void WebViewImpl::SetWindowFeatures(const WebWindowFeatures& features) { 3575 void WebViewImpl::SetWindowFeatures(const WebWindowFeatures& features) {
3576 page_->GetChromeClient().SetWindowFeatures(features); 3576 page_->SetWindowFeatures(features);
3577 } 3577 }
3578 3578
3579 void WebViewImpl::SetOpenedByDOM() { 3579 void WebViewImpl::SetOpenedByDOM() {
3580 page_->SetOpenedByDOM(); 3580 page_->SetOpenedByDOM();
3581 } 3581 }
3582 3582
3583 void WebViewImpl::SetSelectionColors(unsigned active_background_color, 3583 void WebViewImpl::SetSelectionColors(unsigned active_background_color,
3584 unsigned active_foreground_color, 3584 unsigned active_foreground_color,
3585 unsigned inactive_background_color, 3585 unsigned inactive_background_color,
3586 unsigned inactive_foreground_color) { 3586 unsigned inactive_foreground_color) {
(...skipping 541 matching lines...) Expand 10 before | Expand all | Expand 10 after
4128 if (focused_frame->LocalFrameRoot() != MainFrameImpl()->GetFrame()) 4128 if (focused_frame->LocalFrameRoot() != MainFrameImpl()->GetFrame())
4129 return nullptr; 4129 return nullptr;
4130 return focused_frame; 4130 return focused_frame;
4131 } 4131 }
4132 4132
4133 LocalFrame* WebViewImpl::FocusedLocalFrameAvailableForIme() const { 4133 LocalFrame* WebViewImpl::FocusedLocalFrameAvailableForIme() const {
4134 return ime_accept_events_ ? FocusedLocalFrameInWidget() : nullptr; 4134 return ime_accept_events_ ? FocusedLocalFrameInWidget() : nullptr;
4135 } 4135 }
4136 4136
4137 } // namespace blink 4137 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698