Chromium Code Reviews Chromium Code Reviews
Issue 2874783003:
Remove DEBUG_TEXT_AUTOSIZING_ON_DESKTOP altogether. (Closed)
| OLD | NEW |
|---|---|
| 1 /* | 1 /* |
| 2 * Copyright (C) 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All rights | 2 * Copyright (C) 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All rights |
| 3 * reserved. | 3 * reserved. |
| 4 * | 4 * |
| 5 * Redistribution and use in source and binary forms, with or without | 5 * Redistribution and use in source and binary forms, with or without |
| 6 * modification, are permitted provided that the following conditions | 6 * modification, are permitted provided that the following conditions |
| 7 * are met: | 7 * are met: |
| 8 * 1. Redistributions of source code must retain the above copyright | 8 * 1. 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 * 2. Redistributions in binary form must reproduce the above copyright | 10 * 2. Redistributions in binary form must reproduce the above copyright |
| (...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 54 ; | 54 ; |
| 55 } | 55 } |
| 56 | 56 |
| 57 #if OS(WIN) | 57 #if OS(WIN) |
| 58 static const bool kDefaultSelectTrailingWhitespaceEnabled = true; | 58 static const bool kDefaultSelectTrailingWhitespaceEnabled = true; |
| 59 #else | 59 #else |
| 60 static const bool kDefaultSelectTrailingWhitespaceEnabled = false; | 60 static const bool kDefaultSelectTrailingWhitespaceEnabled = false; |
| 61 #endif | 61 #endif |
| 62 | 62 |
| 63 Settings::Settings() | 63 Settings::Settings() |
| 64 #if DEBUG_TEXT_AUTOSIZING_ON_DESKTOP | 64 : text_autosizing_enabled_(false) SETTINGS_INITIALIZER_LIST {} |
| 65 : text_autosizing_window_size_override_(320, 480), | |
|
pdr.
2017/05/10 19:01:16
I think we need to fully remove text_autosizing_wi
Nico
2017/05/10 19:06:30
It's used in a few (not many) tests, e.g. https://
pdr.
2017/05/10 19:32:54
Oohh, that's right. I forgot how those tests work.
| |
| 66 text_autosizing_enabled_(true) | |
| 67 #else | |
| 68 : text_autosizing_enabled_(false) | |
| 69 #endif | |
| 70 SETTINGS_INITIALIZER_LIST { | |
| 71 } | |
| 72 | 65 |
| 73 std::unique_ptr<Settings> Settings::Create() { | 66 std::unique_ptr<Settings> Settings::Create() { |
| 74 return WTF::WrapUnique(new Settings); | 67 return WTF::WrapUnique(new Settings); |
| 75 } | 68 } |
| 76 | 69 |
| 77 SETTINGS_SETTER_BODIES | 70 SETTINGS_SETTER_BODIES |
| 78 | 71 |
| 79 void Settings::SetDelegate(SettingsDelegate* delegate) { | 72 void Settings::SetDelegate(SettingsDelegate* delegate) { |
| 80 delegate_ = delegate; | 73 delegate_ = delegate; |
| 81 } | 74 } |
| (...skipping 24 matching lines...) Expand all Loading... | |
| 106 | 99 |
| 107 void Settings::SetMockScrollbarsEnabled(bool flag) { | 100 void Settings::SetMockScrollbarsEnabled(bool flag) { |
| 108 ScrollbarTheme::SetMockScrollbarsEnabled(flag); | 101 ScrollbarTheme::SetMockScrollbarsEnabled(flag); |
| 109 } | 102 } |
| 110 | 103 |
| 111 bool Settings::MockScrollbarsEnabled() { | 104 bool Settings::MockScrollbarsEnabled() { |
| 112 return ScrollbarTheme::MockScrollbarsEnabled(); | 105 return ScrollbarTheme::MockScrollbarsEnabled(); |
| 113 } | 106 } |
| 114 | 107 |
| 115 } // namespace blink | 108 } // namespace blink |
| OLD | NEW |
